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

Reconsider/refactor computed properties #636

Open
stefandesu opened this issue Nov 5, 2021 · 0 comments
Open

Reconsider/refactor computed properties #636

stefandesu opened this issue Nov 5, 2021 · 0 comments
Labels
cleanup code cleanup, refactoring, testing...
Milestone

Comments

@stefandesu
Copy link
Member

With #633, I found out that some of the computed properties in the application caused a big slowdown for everything else. As a temporary workaround, I'm using a debounced method to update those properties to replace the computed properties. This mitigates the performance issues, but it's not very elegant and it might cause delays in updating the interface. For example, filtering schemes in ConceptSchemeSelection will now take a moment until the debounced method finally gets called. I'm sure there's a better way to do this, but this will probably require some deeper changes to the application.

@stefandesu stefandesu added the cleanup code cleanup, refactoring, testing... label Nov 5, 2021
@stefandesu stefandesu added this to the 2.0.0 milestone Nov 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup code cleanup, refactoring, testing...
Projects
None yet
Development

No branches or pull requests

1 participant