Skip to content

Respect GL_MAX_UNIFORM_BLOCK_SIZE for dlights buffer#1969

Open
slipher wants to merge 1 commit intoDaemonEngine:masterfrom
slipher:maxlights
Open

Respect GL_MAX_UNIFORM_BLOCK_SIZE for dlights buffer#1969
slipher wants to merge 1 commit intoDaemonEngine:masterfrom
slipher:maxlights

Conversation

@slipher
Copy link
Copy Markdown
Member

@slipher slipher commented May 8, 2026

No description provided.

@illwieckz
Copy link
Copy Markdown
Member

illwieckz commented May 8, 2026

Side note: we haven't fixed it yet, but when you open a PR with a branch from a GitHub fork, it doesn't run Azure CI anymore.

@slipher
Copy link
Copy Markdown
Member Author

slipher commented May 8, 2026

OK I fixed it. Now I just need to delete the duplicate version of the pipeline :)

@illwieckz
Copy link
Copy Markdown
Member

Nice!

@slipher
Copy link
Copy Markdown
Member Author

slipher commented May 8, 2026

It was this ("Require a team member's comment"). It's an organization-scope setting so it ought to be fixed too for Crunch and any other repos we have.

image

@illwieckz
Copy link
Copy Markdown
Member

I had also tested that… but it didn't work. Weird. Anyway, thanks for having fixed it!

@illwieckz
Copy link
Copy Markdown
Member

illwieckz commented May 8, 2026

This doesn't fix the realtime lights on ZX C-1190.

Copy link
Copy Markdown
Member

@illwieckz illwieckz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Anyway, more fixes are welcome, LGTM.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants