-
-
Notifications
You must be signed in to change notification settings - Fork 21k
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
Batching+Modulate issue for multimesh drawn in _draw with multiple modulated sprites before it #51883
Comments
Not sure whether it needs to be in |
I'll take a look. 👍 Interesting that it works fine in GLES2, suggests it might be a state or something simple that is out of sync. |
This is already fixed in 3.x. Be sure to try the latest version, you say you are using 3.3 stable, which is the same version as the issue you linked reported the issue in (so it is not surprising you see the same bug). Try the latest 3.3 (3.3.3) or failing that 3.4 beta 4. EDIT: Turns out it is fixed in 3.x branch (i.e. 3.4) but not working in 3.3.3, maybe one of the fixes was not cherry picked. I'll try and work out which. This may need @akien-mga help (maybe when back from holiday) as I'm not that much of a git expert for finding which PRs have been cherry picked to 3.3. Pretty much all the batching PRs should have been ok to cherry pick afaik, so maybe we missed one. |
3.3.stable at the time was 3.3 |
You can filter out PRs which have been cherry-picked by searching for Here's the list of PRs in 3.4 milestone which you made and which haven't been cherry-picked for any 3.3.x release. I don't think anything batching related: There's of course always the option that I might have commented "Cherry-picked for 3.3.3." on a PR but actually forgot to do the cherry-pick locally. The easiest way to find out would be to identify again which PR fixed it in |
Maybe I can also just checkout the relevant files and do a diff to track it down, that may end up being easier. But I'll try this search! 👍 |
Yeah if you can identify what's the difference between the two branches' relevant code, |
Got it I think. Line 869
In the 3.3 version. I remember fixing this, having to find which PR is was. And it's one of clayjohn's, that's why we didn't spot it probably. So this PR just needs cherry picking I think. @akien-mga |
Thanks, I cherry-picked it for 3.3.4, so this can be closed. |
Godot version
3.3.stable
System information
Linux, GLES3, GTX 750Ti (nouveau)
Issue description
An issue similar to #48122 but between two sprites and a
Node2D
using_draw
to draw aMultiMesh
. If I understand correctly, the original issue is supposed to have been fixed already (cherry picked to 3.3 so should affect me).Setup:
The script:
Expected output:
Many
icon.png
s are rendered normally.Actual output:
The
icon.png
s receive the modulate of the sprites above them.It works fine on GLES2 or if batching is disabled.
Steps to reproduce
Run the reproduction project under GLES3 with batching enabled.
Minimal reproduction project
modbatch.zip
The text was updated successfully, but these errors were encountered: