Skip to content

Add secondary DICOM server parameter for IDC viewer handler #24

@fedorov

Description

@fedorov

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 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

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.

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions