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

Some performance improvements. #4054

Closed
wants to merge 7 commits into from
Closed

Conversation

rustbasic
Copy link
Contributor

@rustbasic rustbasic commented Feb 16, 2024

Close #4022

Some performance improvements.

@rustbasic rustbasic closed this Feb 16, 2024
@rustbasic rustbasic reopened this Feb 17, 2024
@emilk
Copy link
Owner

emilk commented Feb 17, 2024

Please elaborate on what is happening here

@rustbasic
Copy link
Contributor Author

@emilk

It's done up to this point.

There are no special bugs.
If you look at the source code, there is no special change from the original intention.

This is what everyone originally intended, and it eliminates commonly used bottlenecks and improves some subtle performance.
It may be difficult to notice performance improvements in simple programs, but performance has improved.

@rustbasic rustbasic changed the title Fix : Unusable When Slowed Down Some performance improvements. Feb 17, 2024
@rustbasic
Copy link
Contributor Author

@emilk

Close #4022

I think issue #4022 can be closed.
FPS is relatively improved at around 40-50.

Reducing FPS from 60 to 50 when creating a viewport is a separate issue.
I think it would be best to end here for now.

Not directly related to this, but influencing,
#4053 FontBook (bulk widget) issue still remains.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Immediate Viewport speed issue
2 participants