-
Notifications
You must be signed in to change notification settings - Fork 37
feat: Implement a cache of script inheritence #314
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
feat: Implement a cache of script inheritence #314
Conversation
There was a clobbered importance order that is no longer being called cosmetically the Inheritance stacks are now being saved so they don't have to be recalced again on each iteration, this saves the majority of the sorting time
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.
Thank You!
I tested it with my 18 mods in Brotato, and the difference is like night and day.
( And still works as expected 👀 )
Comments describing the sorting process and artifcats
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.
Nice! Thanks again for your help. Feel free to merge.
There was a clobbered importance order that is no longer being called cosmetically
the Inheritance stacks are now being saved so they don't have to be recalced again on each iteration, this saves the majority of the sorting time
A ScriptExtensionData type has been removed since it was only used to keep attributes together through the sort and a bit of unnecessary work was done to maintain the type.
closes
#311