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.

Sign up for GitHub

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

Add many command origination effects. #63

Merged
merged 10 commits into from
Jul 3, 2024
Merged

Add many command origination effects. #63

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
fe4991e
Add many more command events. This won't have an effect until we writ…
aduane Jun 22, 2024
6db8146
Use note uuid instead of note id. Not sure I like the attribute renam…
aduane Jun 26, 2024
5546b48
Fix what looks like a copy/paste error in goal.
aduane Jun 26, 2024
237bfe5
WIP: Fix diagnosis command date display issue and start on the diagno…
aduane Jun 26, 2024
e32a590
Update fixture event attribute
aduane Jun 28, 2024
9f3de1b
Move some of the attribute manipulation logic to the effect interpret…
aduane Jun 28, 2024
14cb669
Revert some changes I had made
aduane Jun 29, 2024
4455eb1
Some type changes for the prescribe command.
aduane Jul 2, 2024
ad9dc2c
Ignore this attribute for now
aduane Jul 2, 2024
278780c
Fix tests
aduane Jul 2, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions canvas_generated/messages/effects_pb2.py
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

110 changes: 110 additions & 0 deletions canvas_generated/messages/effects_pb2.pyi
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,12 +13,122 @@ class EffectType(int, metaclass=_enum_type_wrapper.EnumTypeWrapper):
AUTOCOMPLETE_SEARCH_RESULTS: _ClassVar[EffectType]
ADD_BANNER_ALERT: _ClassVar[EffectType]
REMOVE_BANNER_ALERT: _ClassVar[EffectType]
ORIGINATE_ASSESS_COMMAND: _ClassVar[EffectType]
EDIT_ASSESS_COMMAND: _ClassVar[EffectType]
DELETE_ASSESS_COMMAND: _ClassVar[EffectType]
COMMIT_ASSESS_COMMAND: _ClassVar[EffectType]
ENTER_IN_ERROR_ASSESS_COMMAND: _ClassVar[EffectType]
ORIGINATE_DIAGNOSE_COMMAND: _ClassVar[EffectType]
EDIT_DIAGNOSE_COMMAND: _ClassVar[EffectType]
DELETE_DIAGNOSE_COMMAND: _ClassVar[EffectType]
COMMIT_DIAGNOSE_COMMAND: _ClassVar[EffectType]
ENTER_IN_ERROR_DIAGNOSE_COMMAND: _ClassVar[EffectType]
ORIGINATE_GOAL_COMMAND: _ClassVar[EffectType]
EDIT_GOAL_COMMAND: _ClassVar[EffectType]
DELETE_GOAL_COMMAND: _ClassVar[EffectType]
COMMIT_GOAL_COMMAND: _ClassVar[EffectType]
ENTER_IN_ERROR_GOAL_COMMAND: _ClassVar[EffectType]
ORIGINATE_HPI_COMMAND: _ClassVar[EffectType]
EDIT_HPI_COMMAND: _ClassVar[EffectType]
DELETE_HPI_COMMAND: _ClassVar[EffectType]
COMMIT_HPI_COMMAND: _ClassVar[EffectType]
ENTER_IN_ERROR_HPI_COMMAND: _ClassVar[EffectType]
ORIGINATE_MEDICATION_STATEMENT_COMMAND: _ClassVar[EffectType]
EDIT_MEDICATION_STATEMENT_COMMAND: _ClassVar[EffectType]
DELETE_MEDICATION_STATEMENT_COMMAND: _ClassVar[EffectType]
COMMIT_MEDICATION_STATEMENT_COMMAND: _ClassVar[EffectType]
ENTER_IN_ERROR_MEDICATION_STATEMENT_COMMAND: _ClassVar[EffectType]
ORIGINATE_PLAN_COMMAND: _ClassVar[EffectType]
EDIT_PLAN_COMMAND: _ClassVar[EffectType]
DELETE_PLAN_COMMAND: _ClassVar[EffectType]
COMMIT_PLAN_COMMAND: _ClassVar[EffectType]
ENTER_IN_ERROR_PLAN_COMMAND: _ClassVar[EffectType]
ORIGINATE_PRESCRIBE_COMMAND: _ClassVar[EffectType]
EDIT_PRESCRIBE_COMMAND: _ClassVar[EffectType]
DELETE_PRESCRIBE_COMMAND: _ClassVar[EffectType]
COMMIT_PRESCRIBE_COMMAND: _ClassVar[EffectType]
ENTER_IN_ERROR_PRESCRIBE_COMMAND: _ClassVar[EffectType]
ORIGINATE_QUESTIONNAIRE_COMMAND: _ClassVar[EffectType]
EDIT_QUESTIONNAIRE_COMMAND: _ClassVar[EffectType]
DELETE_QUESTIONNAIRE_COMMAND: _ClassVar[EffectType]
COMMIT_QUESTIONNAIRE_COMMAND: _ClassVar[EffectType]
ENTER_IN_ERROR_QUESTIONNAIRE_COMMAND: _ClassVar[EffectType]
ORIGINATE_REASON_FOR_VISIT_COMMAND: _ClassVar[EffectType]
EDIT_REASON_FOR_VISIT_COMMAND: _ClassVar[EffectType]
DELETE_REASON_FOR_VISIT_COMMAND: _ClassVar[EffectType]
COMMIT_REASON_FOR_VISIT_COMMAND: _ClassVar[EffectType]
ENTER_IN_ERROR_REASON_FOR_VISIT_COMMAND: _ClassVar[EffectType]
ORIGINATE_STOP_MEDICATION_COMMAND: _ClassVar[EffectType]
EDIT_STOP_MEDICATION_COMMAND: _ClassVar[EffectType]
DELETE_STOP_MEDICATION_COMMAND: _ClassVar[EffectType]
COMMIT_STOP_MEDICATION_COMMAND: _ClassVar[EffectType]
ENTER_IN_ERROR_STOP_MEDICATION_COMMAND: _ClassVar[EffectType]
ORIGINATE_UPDATE_GOAL_COMMAND: _ClassVar[EffectType]
EDIT_UPDATE_GOAL_COMMAND: _ClassVar[EffectType]
DELETE_UPDATE_GOAL_COMMAND: _ClassVar[EffectType]
COMMIT_UPDATE_GOAL_COMMAND: _ClassVar[EffectType]
ENTER_IN_ERROR_UPDATE_GOAL_COMMAND: _ClassVar[EffectType]
UNKNOWN_EFFECT: EffectType
LOG: EffectType
ADD_PLAN_COMMAND: EffectType
AUTOCOMPLETE_SEARCH_RESULTS: EffectType
ADD_BANNER_ALERT: EffectType
REMOVE_BANNER_ALERT: EffectType
ORIGINATE_ASSESS_COMMAND: EffectType
EDIT_ASSESS_COMMAND: EffectType
DELETE_ASSESS_COMMAND: EffectType
COMMIT_ASSESS_COMMAND: EffectType
ENTER_IN_ERROR_ASSESS_COMMAND: EffectType
ORIGINATE_DIAGNOSE_COMMAND: EffectType
EDIT_DIAGNOSE_COMMAND: EffectType
DELETE_DIAGNOSE_COMMAND: EffectType
COMMIT_DIAGNOSE_COMMAND: EffectType
ENTER_IN_ERROR_DIAGNOSE_COMMAND: EffectType
ORIGINATE_GOAL_COMMAND: EffectType
EDIT_GOAL_COMMAND: EffectType
DELETE_GOAL_COMMAND: EffectType
COMMIT_GOAL_COMMAND: EffectType
ENTER_IN_ERROR_GOAL_COMMAND: EffectType
ORIGINATE_HPI_COMMAND: EffectType
EDIT_HPI_COMMAND: EffectType
DELETE_HPI_COMMAND: EffectType
COMMIT_HPI_COMMAND: EffectType
ENTER_IN_ERROR_HPI_COMMAND: EffectType
ORIGINATE_MEDICATION_STATEMENT_COMMAND: EffectType
EDIT_MEDICATION_STATEMENT_COMMAND: EffectType
DELETE_MEDICATION_STATEMENT_COMMAND: EffectType
COMMIT_MEDICATION_STATEMENT_COMMAND: EffectType
ENTER_IN_ERROR_MEDICATION_STATEMENT_COMMAND: EffectType
ORIGINATE_PLAN_COMMAND: EffectType
EDIT_PLAN_COMMAND: EffectType
DELETE_PLAN_COMMAND: EffectType
COMMIT_PLAN_COMMAND: EffectType
ENTER_IN_ERROR_PLAN_COMMAND: EffectType
ORIGINATE_PRESCRIBE_COMMAND: EffectType
EDIT_PRESCRIBE_COMMAND: EffectType
DELETE_PRESCRIBE_COMMAND: EffectType
COMMIT_PRESCRIBE_COMMAND: EffectType
ENTER_IN_ERROR_PRESCRIBE_COMMAND: EffectType
ORIGINATE_QUESTIONNAIRE_COMMAND: EffectType
EDIT_QUESTIONNAIRE_COMMAND: EffectType
DELETE_QUESTIONNAIRE_COMMAND: EffectType
COMMIT_QUESTIONNAIRE_COMMAND: EffectType
ENTER_IN_ERROR_QUESTIONNAIRE_COMMAND: EffectType
ORIGINATE_REASON_FOR_VISIT_COMMAND: EffectType
EDIT_REASON_FOR_VISIT_COMMAND: EffectType
DELETE_REASON_FOR_VISIT_COMMAND: EffectType
COMMIT_REASON_FOR_VISIT_COMMAND: EffectType
ENTER_IN_ERROR_REASON_FOR_VISIT_COMMAND: EffectType
ORIGINATE_STOP_MEDICATION_COMMAND: EffectType
EDIT_STOP_MEDICATION_COMMAND: EffectType
DELETE_STOP_MEDICATION_COMMAND: EffectType
COMMIT_STOP_MEDICATION_COMMAND: EffectType
ENTER_IN_ERROR_STOP_MEDICATION_COMMAND: EffectType
ORIGINATE_UPDATE_GOAL_COMMAND: EffectType
EDIT_UPDATE_GOAL_COMMAND: EffectType
DELETE_UPDATE_GOAL_COMMAND: EffectType
COMMIT_UPDATE_GOAL_COMMAND: EffectType
ENTER_IN_ERROR_UPDATE_GOAL_COMMAND: EffectType

class Effect(_message.Message):
__slots__ = ("type", "payload", "plugin_name")
Expand Down
44 changes: 25 additions & 19 deletions canvas_sdk/commands/base.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,11 +1,12 @@
import json
import re
from enum import EnumType
from typing import get_args

from pydantic import BaseModel, ConfigDict, model_validator
from typing_extensions import Self

from canvas_sdk.effects import Effect
from canvas_sdk.effects import Effect, EffectType


class _BaseCommand(BaseModel):
Expand All @@ -14,15 +15,18 @@ class _BaseCommand(BaseModel):
class Meta:
key = ""

def constantized_key(self) -> str:
return re.sub(r"(?<!^)(?=[A-Z])", "_", self.Meta.key).upper()

# todo: update int to str as we should use external identifiers
note_id: int | None = None
note_uuid: str | None = None
command_uuid: str | None = None
user_id: int

@model_validator(mode="after")
def _verify_has_note_id_or_command_id(self) -> Self:
if not self.note_id and not self.command_uuid:
raise ValueError("Command should have either a note_id or a command_uuid.")
def _verify_has_note_uuid_or_command_id(self) -> Self:
if not self.note_uuid and not self.command_uuid:
raise ValueError("Command should have either a note_uuid or a command_uuid.")
return self

@property
Expand Down Expand Up @@ -52,7 +56,7 @@ def _get_property_type(cls, name: str) -> type:
@classmethod
def command_schema(cls) -> dict:
"""The schema of the command."""
base_properties = {"note_id", "command_uuid", "user_id"}
base_properties = {"note_uuid", "command_uuid", "user_id"}
schema = cls.model_json_schema()
return {
definition.get("commands_api_name", name): {
Expand All @@ -66,23 +70,25 @@ def command_schema(cls) -> dict:

def originate(self) -> Effect:
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We should run the pydantic validators when these effect methods are called rather than on assignment. This should allow us to set the attributes individually instead of only in the constructor.
Example:

plan = PlanCommand()
plan.narrative = "This is my narrative"
plan.originate()  # <-- only validate here

Copy link
Collaborator

Choose a reason for hiding this comment

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

good idea. want me to tackle that?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Sure, once this gets merged. We should chat about other priorities soon too.

"""Originate a new command in the note body."""
if not self.note_id:
if not self.note_uuid:
raise AttributeError("Note id is required to originate a command")
return {
"type": f"ADD_{self.Meta.key.upper()}_COMMAND",
"payload": {
"user": self.user_id,
"note": self.note_id,
"data": self.values,
},
}
return Effect(
type=EffectType.Value(f"ORIGINATE_{self.constantized_key()}_COMMAND"),
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I changed ADD to ORIGINATE.

payload=json.dumps(
{
"user": self.user_id,
"note": self.note_uuid,
"data": self.values,
}
),
)

def edit(self) -> Effect:
"""Edit the command."""
if not self.command_uuid:
raise AttributeError("Command uuid is required to edit a command")
return {
"type": f"EDIT_{self.Meta.key.upper()}_COMMAND",
"type": f"EDIT_{self.constantized_key()}_COMMAND",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I changed these from self.Meta.key.upper() to a call to the new method constantized_key to account for multi-word command types. Instead of EDIT_MEDICATIONSTATEMENT_COMMAND we want the more readable EDIT_MEDICATION_STATEMENT_COMMAND.

"payload": {
"user": self.user_id,
"command": self.command_uuid,
Expand All @@ -95,7 +101,7 @@ def delete(self) -> Effect:
if not self.command_uuid:
raise AttributeError("Command uuid is required to delete a command")
return {
"type": f"DELETE_{self.Meta.key.upper()}_COMMAND",
"type": f"DELETE_{self.constantized_key()}_COMMAND",
"payload": {"command": self.command_uuid, "user": self.user_id},
}

Expand All @@ -104,7 +110,7 @@ def commit(self) -> Effect:
if not self.command_uuid:
raise AttributeError("Command uuid is required to commit a command")
return {
"type": f"COMMIT_{self.Meta.key.upper()}_COMMAND",
"type": f"COMMIT_{self.constantized_key()}_COMMAND",
"payload": {"command": self.command_uuid, "user": self.user_id},
}

Expand All @@ -113,6 +119,6 @@ def enter_in_error(self) -> Effect:
if not self.command_uuid:
raise AttributeError("Command uuid is required to enter in error a command")
return {
"type": f"ENTER_IN_ERROR_{self.Meta.key.upper()}_COMMAND",
"type": f"ENTER_IN_ERROR_{self.constantized_key()}_COMMAND",
"payload": {"command": self.command_uuid, "user": self.user_id},
}
13 changes: 0 additions & 13 deletions canvas_sdk/commands/commands/diagnose.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,16 +29,3 @@ def values(self) -> dict:
),
"today_assessment": self.today_assessment,
}


# how do we make sure icd10_code is a valid code?

# idea1:
# create an auto-generated enum class of all possible icd10s, then type the field as that enum
# will require releasing a new version with the new codes every year, and devs will need to update
# to make sure they have the latest version to get the right set of codes.

# idea2:
# see if we can get ValueSets to play nicely with pydantic

# idea3: runtime warning after pinging ontologies
2 changes: 1 addition & 1 deletion canvas_sdk/commands/commands/goal.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ def values(self) -> dict:
return {
"goal_statement": self.goal_statement,
"start_date": (self.start_date.isoformat() if self.start_date else None),
"due_date": (self.due_date.isoformat() if self.start_date else None),
"due_date": (self.due_date.isoformat() if self.due_date else None),
aduane marked this conversation as resolved.
Show resolved Hide resolved
"achievement_status": (
self.achievement_status.value if self.achievement_status else None
),
Expand Down
12 changes: 7 additions & 5 deletions canvas_sdk/commands/commands/prescribe.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,11 +18,11 @@ class Substitutions(Enum):

fdb_code: str = Field(json_schema_extra={"commands_api_name": "prescribe"})
icd10_codes: list[str] | None = Field(
None, json_schema_extra={"commandsd_api_name": "indications"}
None, json_schema_extra={"commands_api_name": "indications"}
aduane marked this conversation as resolved.
Show resolved Hide resolved
)
sig: str
days_supply: int | None = None
quantity_to_dispense: Decimal
quantity_to_dispense: Decimal | float | int | None = None
type_to_dispense: str
refills: int
substitutions: Substitutions = Substitutions.ALLOWED # type: ignore
Expand All @@ -38,10 +38,12 @@ def values(self) -> dict:
"icd10_codes": self.icd10_codes,
"sig": self.sig,
"days_supply": self.days_supply,
"quantity_to_dispense": self.quantity_to_dispense,
"type_to_dispense": self.type_to_dispense,
"quantity_to_dispense": str(Decimal(self.quantity_to_dispense))
if self.quantity_to_dispense
else None,
# "type_to_dispense": self.type_to_dispense,
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This breaks the interpreter if sent. Suppressing until the interpreter works.

"refills": self.refills,
"substitutions": self.substitutions,
"substitutions": self.substitutions.value,
"pharmacy": self.pharmacy,
"prescriber_id": self.prescriber_id,
"note_to_pharmacist": self.note_to_pharmacist,
Expand Down
Loading
Loading