-
Notifications
You must be signed in to change notification settings - Fork 0
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
Column configuration #143
Comments
DCW will utilize this via https://www.pivotaltracker.com/story/show/183147834 |
Ordered list. |
@ivan-c to add straw horse config which includes the default fields. |
@achen2401 @pbugni @mcjustin @lober how does something like this look? Happy to make edits (or suggest your own!), but tried to represent "hardcoded" columns (no
|
@achen2401 sorry I kept editing my comment, the expressions I included in my example were JSONPath, not FHIRPath. There are a lot more libraries that implement JSONPath than FHIRPath. However, I took a quick look at the FHIRPath lib you linked, and didn't see any reason it shouldn't work (if my examples were updated to FHIRPath expressions) |
@ivan-c weren't we aiming for the labels to be configurable strings, independent of the reference to the field? It looks like your labels are acting as field references, at least for the "hardcoded" columns (no |
@mcjustin Good point- we wouldn't be able to rename the labels for the hard-coded columns with the example I gave. Maybe we could use some hard-coded constants, one for each hard-coded field, eg:
|
This seems like a difficult way to have a discussion. I'm willing to do it, but it seems like GitHub issues might be best used first to identify and issue then to point to where it is being discussed, related materials, etc. and finally to the documentation of the agreed upon approach to resolve the issue. Not everything is that complex, of course, but I can't figure out what the current example of DASHBOARD_COLUMNS is, what FHIRPath loos like, etc. I'd like to be able to propose some additional column attributes. On a separate note (also not a good place for questions...?!) do FHIRPath expressions always evaluate to the contents of an item in a resource, or can they also evaluate to True or False? I'll avoid tagging anyone for now, and see if these two questions get a replies...! |
@lober I tried out the config structure proposed by Ivan, with the expressions or
I added two additional attributes, i.e.
Currently the expressions evaluate to the contents of an item in a resource, and I don't think they can be evaluated to be true or false, but I could be wrong. |
I do see some boolean operations at https://hl7.org/fhir/fhirpath.html , eg "hasValue", "conformsTo", "memberOf". |
Noting that we didn't decide between these two:
|
@sierramatice You can see a couple configurations of this at https://femr.isacc.dev.cirg.uw.edu and https://femr.helloworld.dev.cirg.uw.edu (test/test). |
@mcjustin I added a mock up and comments to the PT story |
We need a flexible configuration system to define which columns and the FHIR source of each.
A few examples:
MRN - a column that doesn't exist in COSRI, but others will need, and presumably pull from a Patient.identifier with a particular
system
value.Alerts - a source FHIR type that will bring patients to the top of the list with important messages for end users.
The text was updated successfully, but these errors were encountered: