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

Zib-AnatomicalLocation #6

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Zib-AnatomicalLocation #6

wants to merge 14 commits into from

Conversation

ArdonToonstra
Copy link

Initial commit of new profiling proposal based on recent discussions (https://nictiz.atlassian.net/browse/ZIBFHIR2024-173).

@ArdonToonstra ArdonToonstra changed the title New profiling for AnatomicalLocation zib-AnatomicalLocation Oct 1, 2024
@ArdonToonstra
Copy link
Author

ArdonToonstra commented Oct 8, 2024

Reworked the profiles by moving the location binding from CodeableConcept.coding to CodeableConcept. Also added all QA requirements, including release-notes based on a newly created public jira ticket (https://nictiz.atlassian.net/browse/ZIBFHIR-113)

Tested this approach using a dummy profile and multiple examples:

  • Forge does not remove the binding when opened in host profile (it does when you open and resave the AnatomicalLocation profile on CodeableConcept)
  • No validator errors.
  • However, I was not able to trigger a validation error with incorrect values in in the host profile while having a Required binding in the datatype profile. This was both the case for the Java and .NET validator. This might require further investigation.

Also, the QA gives a "Fail: "check nl-core overrides" --> but I don't understand why...

@pieter-edelman-nictiz would you be willing to have another look?

@pieter-edelman-nictiz
Copy link
Member

The QA error is quite trivial; in the nl-core profile, you should override the mention of “zib-” profile to the “nl-core-” profile.

@pieter-edelman-nictiz
Copy link
Member

When generating a snapshot in Forge or using the HL7 Validator, the binding from the dataset profile is ignored and reverts to the default from the base resource.

@ArdonToonstra
Copy link
Author

When generating a snapshot in Forge or using the HL7 Validator, the binding from the dataset profile is ignored and reverts to the default from the base resource.

@pieter-edelman-nictiz , thanks for checking. I will investigate further... I posed this point on zulip. Might want to include this then to our list of things to check. If it doesn't work out in before publication time, we can revert back to the 'old' way and include the binding in the host profiles. Agree?

@LuudSlagter LuudSlagter changed the title zib-AnatomicalLocation Zib-AnatomicalLocation Nov 6, 2024
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.

2 participants