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

OPT.complete() should add the alternative DV_CODED_TEXT node for all DV_TEXT #168

Open
ppazos opened this issue May 23, 2023 · 2 comments
Open
Assignees

Comments

@ppazos
Copy link
Collaborator

ppazos commented May 23, 2023

  1. If a node is DV_TEXT and doesn't have an alternative DV_CODED_TEXT already in the OPT, then add a DV_CODED_TEXT alternative to the attribute.children, that will allow to execute queries over the text or coded text.
  2. The new DV_CODED_TEXT node should be open (no constraints added).
  3. Note this will add coded nodes for all the LOCATABLE.name attributes, that is not an issue, though we can add an option to avoid it.
@ppazos ppazos self-assigned this May 23, 2023
@ppazos
Copy link
Collaborator Author

ppazos commented May 24, 2023

NOTE: it seems if I have coded text, it allows text but not viceversa

@ppazos
Copy link
Collaborator Author

ppazos commented May 24, 2023

NOTE: we already have this implemented in EHRServer ppazos/cabolabs-ehrserver#748 in the OperationalTemplateIndexerService, we might need to retest this with specific OPTs to see why the alternative coded is not appearing as a query path.

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