Optimize Projection::determinant()
#103671
Open
+26
−12
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
It would be good to mention in the comment what algorithm is used. |
Added the wiki link from the proposal |
|
Static checks fail because of code formatting. But after clang-format the code doesn't look as clear, maybe it makes sence to describe code briefly in comment, not just give link to wiki. |
updated the comment and used clang-format. |
ProfessorOctopus
added a commit
to ProfessorOctopus/godot
that referenced
this pull request
Mar 26, 2025
Fix crash when removing breakpoints from DAP, and multiple fixes godotengine#104523 by rsubtil TextureEditors: Compile shader/material only once godotengine#104488 by arkology Renderer: Warn when images need to be converted due to their formats being unsupported by hardware godotengine#104480 by BlueCube3310 Optimize startup times by using ubrk_clone instead of ubrk_open. godotengine#104455 by Ivorforce Pause audio when game is paused godotengine#104420 by pafuent Optimize reverb by removing stray volatile from the undenormalize function signature. godotengine#104239 by Ivorforce Add Memory::alloc_static_zeroed to allocate memory that's filled with zeroes. godotengine#104124 by Ivorforce Switch occlusion culling to be based on depth instead of Euclidean distance godotengine#103798 by Rudolph-B Optimize Object::cast_to by assuming no virtual and multiple inheritance, gaining 7x throughput over dynamic_cast. godotengine#103708 Ivorforce Optimize Projection::determinant() godotengine#103671 by MaxIsJoe Reuse Sprite3D meshes across nodes when possible. godotengine#103312 by chocola-mint Reduce frequency of save_default_environment() calls godotengine#102991 by KoBeWi Optimize GDScriptLambdaCallable by skipping the unnecessary ObjectDB lookup for script. godotengine#102930 by Ivorforce
ProfessorOctopus
added a commit
to ProfessorOctopus/godot
that referenced
this pull request
Apr 3, 2025
Optimize Projection::determinant() godotengine#103671 Co-Authored-By: MaxIsJoe <34368774+maxisjoe@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Implements ( godotengine/godot-proposals#11821 )