Skip to content

Respect GL_MAX_UNIFORM_BLOCK_SIZE for dlights buffer#1969

Merged
slipher merged 1 commit into
DaemonEngine:masterfrom
slipher:maxlights
May 9, 2026
Merged

Respect GL_MAX_UNIFORM_BLOCK_SIZE for dlights buffer#1969
slipher merged 1 commit into
DaemonEngine:masterfrom
slipher:maxlights

Conversation

@slipher

@slipher slipher commented May 8, 2026

Copy link
Copy Markdown
Member

No description provided.

@illwieckz

illwieckz commented May 8, 2026

Copy link
Copy Markdown
Member

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

slipher commented May 8, 2026

Copy link
Copy Markdown
Member Author

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

@illwieckz

Copy link
Copy Markdown
Member

Nice!

@slipher

slipher commented May 8, 2026

Copy link
Copy Markdown
Member Author

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

illwieckz commented May 8, 2026

Copy link
Copy Markdown
Member

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

@illwieckz illwieckz left a comment

Copy link
Copy Markdown
Member

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.

@slipher slipher merged commit 0cddd7e into DaemonEngine:master May 9, 2026
14 checks passed
@slipher slipher deleted the maxlights branch May 9, 2026 19:48
@illwieckz illwieckz mentioned this pull request May 9, 2026
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