Skip to content

Make DESCRIPTION EVENT SERVICE optional.#532

Open
PaulMartinsen wants to merge 2 commits into
masterfrom
451-remove-the-description-event-service-from-the-minimum-requirements-imposed-by-sdpi
Open

Make DESCRIPTION EVENT SERVICE optional.#532
PaulMartinsen wants to merge 2 commits into
masterfrom
451-remove-the-description-event-service-from-the-minimum-requirements-imposed-by-sdpi

Conversation

@PaulMartinsen
Copy link
Copy Markdown
Collaborator

@PaulMartinsen PaulMartinsen commented May 21, 2026

📑 Description

Removed the DESCRIPTION EVENT SERVICE from list of minimum required services following brief discussion of #451 at PaT#23.

Providers may still implement and offer this service, but aren't required to do so. Consumers that observe no DESCRIPTION EVENT SERVICE is offered may be confident that the provider won't be changing its medical device description at inconvenient times.

☑ Mandatory Tasks

The following aspects have been respected by the pull request assignee and at least one reviewer:

  • Changelog update (necessity checked and entry added or not added respectively)
  • Pull Request Assignee
  • Reviewer

@PaulMartinsen PaulMartinsen self-assigned this May 21, 2026
[[payload_dev_25_get_metadata_biceps_services]]BICEPS Services:: Collection of BICEPS services the <<vol1_spec_sdpi_p_actor_somds_provider>> offers, including but not limited to one or multiple of the following BICEPS services (<<ref_ieee_11073_10207_2017>> Section 7.3 Service Model):
* GET SERVICE (mandatory)
* SET SERVICE
* DESCRIPTION EVENT SERVICE
Copy link
Copy Markdown
Collaborator

@d-gregorczyk d-gregorczyk May 22, 2026

Choose a reason for hiding this comment

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

I am a little confused as to where the description event service is mandated here. Reading the previous sentence to this enumeration, it says including but not limited to one or multiple of the following BICEPS services. Mandatory in this list is the GET SERVICE, the others are not stated as "to be available at all times".

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

As far as I can tell, the only occurrence to be removed is here: "Table 2:A.1-1. Minimum required port types" or did I miss something?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

When read with the immediately following note, it appears the bulleted list here reads as mandatory services and the note talks about the ones that aren't mandatory. So for clarity I removed the description event service list from the bulleted list and added to the note that "There is no mandatory support for provision of...".

Copied from the original text, before this PR is applied:
image

Copy link
Copy Markdown
Collaborator

@d-gregorczyk d-gregorczyk May 22, 2026

Choose a reason for hiding this comment

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

Ok, I see. Maybe we rephrase the whole paragraph by adding Localization Service to the list and justify the missing containment tree service and archive service with obsolescence. Something along the lines:

The CONTAINMENT TREE SERVICE and ARCHIVE SERVICE are missing from this list as the CONTAINMENT SERVICE is insufficiently defined and the ARCHIVE SERVICE is superseded by the History Service.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

But honestly, we can erase the entire note tbh

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

Remove the description event service from the minimum requirements imposed by SDPi.

2 participants