Skip to content

CP ITI 1322#575

Merged
lukeaduncan merged 5 commits intoIHE:ITI-CP-Integrationfrom
oliveregger:cp-iti-1322
Oct 7, 2025
Merged

CP ITI 1322#575
lukeaduncan merged 5 commits intoIHE:ITI-CP-Integrationfrom
oliveregger:cp-iti-1322

Conversation

@oliveregger
Copy link
Copy Markdown
Contributor

Incorporates CP 1322, see https://docs.google.com/document/d/1_akpDRB5dXZands28SUM7suXdTGaCBwQ/edit

The MHDS part will be in PR for MHDS

@slagesse-epic could you add a review

@oliveregger oliveregger changed the base branch from master to ITI-CP-Integration August 15, 2025 16:28
@slagesse-epic
Copy link
Copy Markdown
Member

During balloting, it was discussed whether these should be integrated into the TF or into the MHD, MHDS, and PDQm supplements as changes to other specifications, since MHD, MHDS, and PDQm are not yet final text. It seems that the general consensus was that these should go into the supplements.

@MaryLJ MaryLJ requested a review from slagesse-epic September 4, 2025 23:22
@JohnMoehrke
Copy link
Copy Markdown
Contributor

You can't put Trial-Implementation updates into the Technical-Framework. Thus this CP must be made to the MHDS. Yes this means that these other profile updates are added as editor-instructions in the MHDS to a page that does not exist today "Changes to other profiles"

@slagesse-epic
Copy link
Copy Markdown
Member

I think the bulk of this is actually a better fit for MHD, since we can talk about how to translate between XCA and MHD without talking about MHDS.

@JohnMoehrke
Copy link
Copy Markdown
Contributor

The CP is not about MHD, it is about how to use XCA to federate MHDS communities.

MHD and XCA have been supported directly from the beginning.

@slagesse-epic
Copy link
Copy Markdown
Member

...for example, the new section 18.2.3.3 describes grouping behavior between XCA and MHD and how to map the queries. I would expect that text to be useful to anyone implementing an XCA Responding Gateway grouped with an MHD document consumer regardless of whether the responding community implements MHDS.

@JohnMoehrke
Copy link
Copy Markdown
Contributor

...for example, the new section 18.2.3.3 describes grouping behavior between XCA and MHD and how to map the queries. I would expect that text to be useful to anyone implementing an XCA Responding Gateway grouped with an MHD document consumer regardless of whether the responding community implements MHDS.

This is a case of a Responding Gateway using an MHD Document Consumer to access the MHDS community (XCA->MHD->MHDS). This is because the ONLY api we have to MHDS is MHD. This is only needed in the case where the community is MHDS.

If the community is XDS this is explained in 18.2.3.2 using an XDS Document Consumer (the logical equivalent)

Are you suggesting that someone would use XCA->MHD->XDS? I guess they could do that, but that is not what we are describing in this CP. This kind of a configuration would likely be lossy in ways that are not necessary.

@slagesse-epic
Copy link
Copy Markdown
Member

I am suggesting that someone might use XCA->MHD, where the community does not implement MHDS. For example, a standalone MHD Document Responder exists in a community with no other IHE integration profiles.

@JohnMoehrke
Copy link
Copy Markdown
Contributor

I am suggesting that someone might use XCA->MHD, where the community does not implement MHDS. For example, a standalone MHD Document Responder exists in a community with no other IHE integration profiles.

I guess that is possible. It was not part of the scope of the CP.

The reason I am pushing back is because many people have incorrectly thought that MHD didn't support XCA on the client side.

@oliveregger
Copy link
Copy Markdown
Contributor Author

I added the CP changes to the other profiles directly to MHDS as suggested, put this PR back go draft that we could reuse it when MDHS gets final text.

@oliveregger oliveregger marked this pull request as draft September 8, 2025 21:13
@oliveregger
Copy link
Copy Markdown
Contributor Author

@JohnMoehrke
Copy link
Copy Markdown
Contributor

JohnMoehrke commented Sep 9, 2025

don't forget to add a note to the top of XCA, and XCPD (in publications repo) indicating that it has changes found in MHDS. (See how XCA deferred response option is mentioned there). This should be done to any Final-Text that has integrated CPs that affect it, but where that integration is in a supplement.

So, this PR could be changed to just doing that.

@oliveregger
Copy link
Copy Markdown
Contributor Author

@slagesse-epic can you review again?

@oliveregger oliveregger marked this pull request as ready for review October 4, 2025 12:33
Copy link
Copy Markdown
Member

@slagesse-epic slagesse-epic left a comment

Choose a reason for hiding this comment

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

Looks good!

@lukeaduncan lukeaduncan merged commit 207b3d3 into IHE:ITI-CP-Integration Oct 7, 2025
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.

4 participants