-
Notifications
You must be signed in to change notification settings - Fork 6
Description
This is follow up on vkt1414/slicer-idc-viewer#2.
@pieper I thought about this after the discussion yesterday. I understand the use case better now, and I think adding the feature that would allow parameterizing slicer://idc-viewer/
with a DICOMweb endpoint is a specific functionality we could explore towards implementing the workflow in the issue above. I initially thought it would be better to have this handler in mpReview, but since most of the logic will be in IDCBrowser, it makes more sense to implement it here.
I think the functionality would be the following:
idc-viewer
handler is passed a list ofSeriesInstanceUID
s as one parameter- and the GCP DICOMweb URL as the second parameter
- handler uses
idc-index
to resolveStudyInstanceUID
for each of theSeriesInstanceUID
s and fetches all of the available series in those studies using DICOMweb from the secondary server
We should reuse the functionality from SlicerDICOMWeb, similar to how it is done in mpReview in https://github.com/SlicerProstate/mpReview/blob/master/mpReview.py#L534-L543 and https://github.com/SlicerProstate/mpReview/blob/master/mpReview.py#L1814.
Is this consistent with what you were looking for @pieper? I was confused by the other issue since it was very broad.
I think we should look into this only after the basic handler is functional.