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

use recorded-date by timerestriction #188

Open
KutSaleh opened this issue Aug 7, 2024 · 7 comments
Open

use recorded-date by timerestriction #188

KutSaleh opened this issue Aug 7, 2024 · 7 comments

Comments

@KutSaleh
Copy link

KutSaleh commented Aug 7, 2024

If we use the timerestriction when querying for diagnoses in a cohort, then the onset-date is used, would it be possible to use the recorded-date in addition as well?

@juliangruendner
Copy link
Collaborator

juliangruendner commented Aug 7, 2024

@KutSaleh additionally is difficult - its still an ongoing discussion which field should be used. In this case its meant to be the clinical date and for most fhir resources this is defined, see: https://hl7.org/fhir/R4/searchparameter-registry.html (search for clinical-date).
Condition is the one exception and there is no clear agreement on what the clinical data should be, so before we search for multiple or go back to recorded-date - a mii wide decisiion has to be made on this issue
@julsas - is there a definitiv answer to this now?

@KutSaleh
Copy link
Author

KutSaleh commented Aug 7, 2024

thank you for your quick reply. I would prefer the recodedDate as this is with 1..1 cardinality in the diagnosis MII module, see https://simplifier.net/mii-basismodul-diagnose-2024/mii_pr_diagnose_condition as well it is listed as a search parameter (recoded-date)

@DannyAmmon
Copy link
Member

Please keep in mind that Condition.onsetDateTime does not exist in structured form in most primary documentation systems.

@KutSaleh
Copy link
Author

is there anything new?

@julsas
Copy link
Member

julsas commented Aug 27, 2024

No final answer but I don't mind switching back to recordedDate.

Condition.onset was chosen because we assumed researchers would be most interested in onset and less interested on recordedDate. Downside is that onset is optional in our profile because most primary systems won't have it present.
Condition.recordedDate will deliver more results as primary systems will have it and it is mandatory in our diagnosis profile.

@KutSaleh
Copy link
Author

Thanx, can it then be changed to recordedDate? or should the final answer be given first? If so, who can give the final answer?

@julsas
Copy link
Member

julsas commented Sep 18, 2024

Yes, it will be changed to recordedDate.

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

No branches or pull requests

4 participants