-
-
Notifications
You must be signed in to change notification settings - Fork 21.5k
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
Fix deprecated mesh conversion #68428
Conversation
re: @clayjohn @The-O-King |
ae1f6ed
to
5a3b1cc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks mostly good, only non-compressed non-octahedral tangents needs to be updated.
I left a comment on each line to show which entry from 3.x it matches for future reference.
Once the non-compressed non-octahedral tangent code is fixed this should be good to merge
The code looks great! The final step before merging is to squash all the commits together so that the whole PR only contains 1 big commit with all your changes. We like to merge one commit at a time to keep the git history clean and navigable. If you don't know how to do that, we have a helpful tutorial in the official documentation https://docs.godotengine.org/en/latest/community/contributing/pr_workflow.html#the-interactive-rebase |
I'm not going to be near a computer for the next week. Can you please squash and merge? |
Sorry, no. We don't have that enabled for this repository :( |
d5fea3a
to
5683aa2
Compare
apparently GitHub codespaces is free now and works on ipad safari, so there you go. |
Thanks! |
In #60309, when octahedral mesh conversion was added to Godot 4.x, the src_offset increment were modified erroneously and changed the increment value.
https://github.com/godotengine/godot/pull/60309/files#r1017101020
https://github.com/godotengine/godot/pull/60309/files#r1017104192
This restores the previous offset values and adds clarifying comments where necessary.