Skip to content
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.

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

Enabled up to 65535 pieces per model #1835

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

Enabled up to 65535 pieces per model #1835

wants to merge 4 commits into from

Conversation

lhog
Copy link
Collaborator

@lhog lhog commented Dec 19, 2024

Make it the least intrusive way, shaders should be backward compatible to the 255 pieces (on sane GPU drivers).
It might even work, needs testing.

Make it the least intrusive way, shaders should be backward compatible to the 254 pieces (on sane GPU drivers).
It might even work, needs testing.
@lhog lhog changed the title Enabled up to 65534 pieces per model Enabled up to 65535 pieces per model Dec 19, 2024
@lhog lhog added game compat Requires a look into how existing games do something or send them a patch status: candidate PRs that should be good to go or important for next release labels Dec 19, 2024
rts/Rendering/GL/myGL.h Outdated Show resolved Hide resolved
Copy link
Collaborator

@sprunk sprunk left a comment

Choose a reason for hiding this comment

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

Looks alright

@saurtron
Copy link
Collaborator

saurtron commented Dec 20, 2024

It might even work, needs testing.

Run the game with this and seems to work fine. Do you expect this kind of testing (ie, just testing if it doesn't break previous behaviour) or do you actually need a unit with more than 256 pieces to validate the new behaviour is correct?

If so, we should look for someone who can prepare that, otherwise maybe the candidate label is a bit optimistic since imo that should be reserved for PRs ready to merge (could create and use candidate-next label, I think that one could be used for things we want to prioritize but not 100% ready yet or needing testing).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
game compat Requires a look into how existing games do something or send them a patch status: candidate PRs that should be good to go or important for next release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants