add sdpi p history service option#452
Conversation
…transaction summary anchors.
… technologies such that protocol buffers.
|
Suggest changing R0512 to something like:
Possible need to tweak R0514 to something like:
Reasoning:I was thinking more about the case where a provider starts providing history information but some of the history becomes unavailable before it is delivered. This could occur when:
Some available options :
I quite like option 3 because:
If the consumer didn't request a baseline mdib, we could sensibly fallback to option 1: end the subscription. Note Added to review. |
|
I would quite like to be able to omit waveform streams from the history because:
But the general case, where providers may omit any information, seems quite complicated and could be problematic. That is, important context may be lost. Could the provider be allowed to omit waveform streams either:
Or maybe there is no value in waveform streams in the historic record and they could just be omitted completely? That is, remove The main problem seems to be missing state versions (R1005), although I think we need to get rid of that one anyway. Note This is addressed in R0516 now |
|
The sequence diagrams weren't showing up for me. It looks like the action isn't copying them into the supplement from Note Works now. |
PaulMartinsen
left a comment
There was a problem hiding this comment.
I've focused on changes to improve clarity and provide a good starting point for trial implementation.
|
|
||
| NOTE: A <<vol1_spec_sdpi_p_actor_somds_provider>> may respond with data that exceed the interval of ++[hm:VersionRange/@StartVersion, hm:VersionRange/@EndVersion]++. | ||
|
|
||
| NOTE: A <<vol1_spec_sdpi_p_actor_somds_provider>> may respond with a subset of data from the interval ++[hm:VersionRange/@StartVersion, hm:VersionRange/@EndVersion]++, e.g., if it cannot fully serve the request due to discarded historic information or filter request that point into the future. |
There was a problem hiding this comment.
For clarity:
… or filter request that point into the future.
… or the filter requests reports from the future.
| **** | ||
| [NORMATIVE] | ||
| ==== | ||
| If it is intended for a <<vol1_spec_sdpi_p_actor_somds_provider>> to include <<term_historic_localized_text>>s in hm:ChangeSequenceReport elements, for an <<term_mdib_sequence_of_changes>> that is requested from a <<vol1_spec_sdpi_p_actor_somds_provider>>, the <<vol1_spec_sdpi_p_actor_somds_provider>> shall transmit filtered <<term_historic_localized_text>>s before transmitting the hm:HistoricMdib or hm:HistoricReport elements that reference the <<term_historic_localized_text>>s. |
There was a problem hiding this comment.
simplify for clarity:
The <<vol1_spec_sdpi_p_actor_somds_provider>> transmitting <<term_historic_localized_text>> shall transmit filtered <<term_historic_localized_text>>s before transmitting hm:HistoricMdib or hm:HistoricReport elements that reference the <<term_historic_localized_text>>s.
Note:
This ensures the localized text is available to the <<vol1_spec_sdpi_p_actor_somds_consumer>> before it is used by other elements.
Made parallel notes in R0515 and R0516 more consistent for clarity.
…nce report header element.
…ption' into 360-add-sdpi-p-history-service-option
…ption' into 360-add-sdpi-p-history-service-option
☑ Mandatory Tasks
The following aspects have been respected by the pull request assignee and at least one reviewer: