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

ihe_iua extension example does not conform to text? #76

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

oliveregger
Copy link
Contributor

No description provided.

Copy link
Contributor

@JohnMoehrke JohnMoehrke left a comment

Choose a reason for hiding this comment

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

I don't know this well enough. I added Joost and Martin to review.

@msmock
Copy link
Contributor

msmock commented Mar 30, 2021

I agree to the proposed change.

Copy link
Contributor

@msmock msmock left a comment

Choose a reason for hiding this comment

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

I agree to the change.

Copy link
Contributor

@joostreuzel joostreuzel left a comment

Choose a reason for hiding this comment

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

The changes make role and purpose singular. Multiplicity was indeed not well defined. Would singular use not be too restrictive? A subject may have several roles in an organization context. Similarly, a token may be used for several purposes? As an alternative we may clarify in the spec the multiplicity of the fields?

@oliveregger
Copy link
Contributor Author

oliveregger commented Apr 21, 2021

The changes make role and purpose singular. Multiplicity was indeed not well defined.

I read the requesting role and purpose singular (see #75), the representation as Coding is also singular (otherwise it would need to be a CodeableConcept). This PR tries to bring in the example congruent to text.

Would singular use not be too restrictive? A subject may have several roles in an organization >context. Similarly, a token may be used for several purposes? As an alternative we may clarify in the >spec the multiplicity of the fields?

According to https://profiles.ihe.net/ITI/TF/Volume2/ITI-40.html I intepret purposeOfUse as 0..1 but subject-roles could indeed be mutiple values.

@joostreuzel
Copy link
Contributor

This PR tries to bring in the example congruent to text.

That is well-understood. However, to align the example with the text, augmentations can be made in two directions: we can improve the example or improve the text...

My proposal would be to:

  • keep the purposeOfUse singular, as in line with ITI40, and with a corrected example as suggested by @oliveregger
  • allow for multiple roles. I think we can encode the roles as an array of Coding as was done in the original example. We would need to update the text to clarify this. I don't think a CodeableConcept is needed as that would bring something new to the table: a text representation of a role (which would be singular again).

@JohnMoehrke
Copy link
Contributor

@jlamy -- please weigh in on this. i think we had a similar problem observed in CareQuality. Although I think that one was SAML (XUA), the solutions should be as similar as we can make them.

@JohnMoehrke
Copy link
Contributor

Note to ALL -- IUA is now published. so this issue must come to a conclusion using the CP system. We can fully document it in an Issue with a Pull-Request. but our final decision needs to go into a CP ballot.

@joostreuzel
Copy link
Contributor

@JohnMoehrke What would be your proposal then? Update the non-normative example as suggested by @oliveregger, and open a seperate issue to update the multiplicity and have that go through the ballot?

@JohnMoehrke
Copy link
Contributor

IHE supplements are published as a single publication. There is no difference in how a normative change is done vs informative. Even Spelling fixes are documented in a CP.

So, this issue and pull-request are the proper mechanism for specific details.... What I am simply pointing out is that the pull-request can NOT be merged until the change goes thru a proper CP ballot. For that CP ballot, we will need a simple WORD document CP pointing at this Issue and Pull-Request, so that the governance of the CP ballot can progress.

@oliveregger
Copy link
Contributor Author

Just wanted to add a CP for this, but have seen that @lynnfel already has done it with CP-ITI-IUA-Issue-75, thanks a lot!

@oliveregger
Copy link
Contributor Author

  • allow for multiple roles. I think we can encode the roles as an array of Coding as was done in the original example. We would need to update the text to clarify this. I don't think a CodeableConcept is needed as that would bring something new to the table: a text representation of a role (which would be singular again).

@joostreuzel agree, an array of Coding would be better, a CodeableConcept is the wrong approach since the codings in there would not have the same semantic.

@oliveregger
Copy link
Contributor Author

@joostreuzel and @msmock this CP will be discussed tomorrow at ITI - CP Processing, do you have time to join? Thursday, May 13th, 12:00:00 PM, ITI - CP Processing - Time - Central Daylight Time (GMT-5)
https://himss.zoom.us/j/91068642326

@lynnfel
Copy link
Contributor

lynnfel commented May 18, 2021

CP discussion determined that subject_role and purpose of use may be encoded with multiple values. Syntax details to be written...

The CP discussion on 18-May-2021 determined that the resolution will be determined in the context of this pull request , and when the solution is determined, it will be represented in CP-ITI-1255 for the purpose of CP balloting.

@JohnMoehrke
Copy link
Contributor

CP is still in progress

@JohnMoehrke JohnMoehrke marked this pull request as draft October 4, 2022 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants