You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@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 of SeriesInstanceUIDs as one parameter
and the GCP DICOMweb URL as the second parameter
handler uses idc-index to resolve StudyInstanceUID for each of the SeriesInstanceUIDs and fetches all of the available series in those studies using DICOMweb from the secondary server
No, we don't. I think we can start discussing this whenever @vkt1414 has the PR opened. But for now I want to focus on TotalSegmentator project and make a push to try to get it done before the PW. I think we can use PW project for ironing out details like the schema and this new functionality.
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 parameteridc-index
to resolveStudyInstanceUID
for each of theSeriesInstanceUID
s and fetches all of the available series in those studies using DICOMweb from the secondary serverWe 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.
The text was updated successfully, but these errors were encountered: