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

Make fast version of SymptomManager #1601

Open
tbhallett opened this issue Feb 28, 2025 · 0 comments
Open

Make fast version of SymptomManager #1601

tbhallett opened this issue Feb 28, 2025 · 0 comments

Comments

@tbhallett
Copy link
Collaborator

In profiling, a lot of time is taken in SymptomManager. This is because it is designed to do a lot of checking (that the symptom exists, and the disease is allowed to apply it, and when its resolved its only resolved by the disease that caused it).

Whilst these checks are useful in development, now that we never have problems, we can dispense with these checks.

We should we create a drop-in replacement for SymptomManager that has the same functionality and interface, but internally does the simplest and quickest thing possible. This could be making all the symptoms columns bools

@tamuri tamuri changed the title Make Fast Version of Symptom Manager Make fast version of SymptomManager Feb 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant