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

[Master] Stability - Math Optimization #959

Closed
wants to merge 1 commit into from

Conversation

RIH-Cellegen
Copy link

@RIH-Cellegen RIH-Cellegen commented Feb 7, 2025

Optimized calculations for most Math related scripts that run more frequently.

Not yet fully tested, waiting for test results and further improvements, however so far it can be experimented with.

(Post edit: Comparison is wrong, will re-investigate the pull request, until then it'll be closed)

The current tests comparing Redot 4.3.1 Alpha 1 (Left Picture) and This version - Redot Master branch (Right Picture)

(3D Scene - Same Project - Simultanious comparision)

image1

(3D Scene - Overdraw Perspective - Same Project - Simultanious comparision)

image2

Optimized calculations for most Math related scripts that run more frequently.
Results show a 20% increase in overall performance on 3D / 2D Scenes.

Not yet fully tested, waiting for test results and further improvements, however so far it can be experimented with.
@RIH-Cellegen RIH-Cellegen changed the title 20% Performance Increase - Math Optimization [Master] 20% Performance Increase - Math Optimization Feb 7, 2025
@Spartan322
Copy link
Member

Spartan322 commented Feb 7, 2025

You shouldn't be comparing 4.3 and master, a very large number of optimizations were already made to master, its only accurate to compare pre-PR vs. post-PR performance. Also you must run pre-commit.

@RIH-Cellegen RIH-Cellegen changed the title [Master] 20% Performance Increase - Math Optimization [Master] Stability - Math Optimization Feb 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants