Skip to content

Only update PushBuffer if any uniform has actually changed #1685

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
VReaperV wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Only update PushBuffer if any uniform has actually changed #1685

VReaperV wants to merge 4 commits into from
+134 −400

Conversation

@VReaperV
Copy link
Contributor

@VReaperV VReaperV commented Jun 23, 2025

Requires #1684

Further improvement for PushBuffer, only update it if it's actually required.

@VReaperV VReaperV added T-Improvement Improvement for an existing feature A-Renderer labels Jun 23, 2025
@VReaperV VReaperV force-pushed the push-buffer-update branch 2 times, most recently from bec3aa0 to ad58d5c Compare June 24, 2025 17:42
@VReaperV
Copy link
Contributor Author

VReaperV commented Jun 24, 2025

Also added a fix for globalFog fog texture.

@VReaperV VReaperV force-pushed the push-buffer-update branch 2 times, most recently from f67e29b to e84d583 Compare June 25, 2025 05:55
@VReaperV
Copy link
Contributor Author

VReaperV commented Jun 25, 2025 •
edited
Loading

Changed the initial commit and added a new one, now it mutualises currentValue and uniformFirewall (in a way that works with any kind of uniform unlike uniformFirewall, and with both buffers and shader programs unlike currentValue), GLUniform.WriteToBuffer() now only has 1 definition, and the uniform setting functions are now much simpler. Also NUKED a bunch of now unused code.

@VReaperV VReaperV force-pushed the push-buffer-update branch from 80717b6 to 8832f45 Compare June 30, 2025 15:27
@VReaperV VReaperV mentioned this pull request Jul 25, 2025
Merged
@VReaperV VReaperV force-pushed the push-buffer-update branch 3 times, most recently from fedb60c to fc2fff3 Compare August 1, 2025 22:59
@VReaperV VReaperV force-pushed the push-buffer-update branch from fc2fff3 to bf51f59 Compare August 19, 2025 11:07
This was referenced Aug 27, 2025
Closed
Merged
@VReaperV VReaperV force-pushed the push-buffer-update branch from 2b38e3f to 857f94f Compare January 9, 2026 01:23
@VReaperV VReaperV force-pushed the push-buffer-update branch from 9a82e42 to 553f6d9 Compare January 9, 2026 01:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

A-Renderer T-Improvement Improvement for an existing feature

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@VReaperV