Skip to content

Reviewing symptom status manager PR#41

Closed
erik-rosenstrom wants to merge 9 commits intopgv5_clinical_statusfrom
etr_pgv5_clinical_status_refactor
Closed

Reviewing symptom status manager PR#41
erik-rosenstrom wants to merge 9 commits intopgv5_clinical_statusfrom
etr_pgv5_clinical_status_refactor

Conversation

@erik-rosenstrom
Copy link
Collaborator

No description provided.

@erik-rosenstrom erik-rosenstrom changed the title Reviewing the PR Reviewing symptom status manager PR Mar 9, 2026
@eqmooring eqmooring force-pushed the pgv5_clinical_status branch from abea006 to 1be24fd Compare March 9, 2026 18:12
default_const = SymptomStatus::NoSymptoms
);

fn plan_symptom_transition(
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See my comments here to motivate distinguishing symptom progression from symptom resolution instead of lumping them into one function for evaluating the transition

@erik-rosenstrom
Copy link
Collaborator Author

Closing. I will rereview #25 when requested

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.

3 participants