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

Performance regression #59

Open
BenSchZA opened this issue Sep 8, 2022 · 0 comments
Open

Performance regression #59

BenSchZA opened this issue Sep 8, 2022 · 0 comments
Labels
enhancement New feature or request

Comments

@BenSchZA
Copy link
Collaborator

BenSchZA commented Sep 8, 2022

In radCAD version 0.11.0 (see CHANGELOG.md) an update was made to ensure that unintended state updates between state update functions within a single partial state update block can not be made. As a result, some models may perform worse - typically models with lots of states and state updates or with large objects in state.

There is a compromise to be made between optimising performance vs. ensuring immutability - TBC what the best solution is. It's likely that it should be handled on a case by case basis. For example, see radCAD deepcopy and deepcopy_method options.

744b5f4

@BenSchZA BenSchZA added the enhancement New feature or request label Sep 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant