large-image-source-dicom 1.28.2__tar.gz → 1.28.2.dev2__tar.gz

{large-image-source-dicom-1.28.2 → large-image-source-dicom-1.28.2.dev2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: large-image-source-dicom
-Version: 1.28.2
+Version: 1.28.2.dev2
 Summary: A DICOM tilesource for large_image.
 Home-page: https://github.com/girder/large_image
 Author: Kitware, Inc.
@@ -17,11 +17,10 @@ Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Requires-Python: >=3.8
 License-File: LICENSE
-Requires-Dist: large-image>=1.28.2
+Requires-Dist: large-image>=1.28.2.dev2
 Requires-Dist: wsidicom>=0.9.0
 Provides-Extra: girder
-Requires-Dist: girder-large-image>=1.28.2; extra == "girder"
-Requires-Dist: girder>=3.2.3; extra == "girder"
+Requires-Dist: girder-large-image>=1.28.2.dev2; extra == "girder"
 
 A DICOM tilesource for large_image.
 

{large-image-source-dicom-1.28.2 → large-image-source-dicom-1.28.2.dev2}/large_image_source_dicom/assetstore/dicomweb_assetstore_adapter.py

@@ -1,5 +1,3 @@
-import json
-
 import cherrypy
 import requests
 from large_image_source_dicom.dicom_tags import dicom_key_to_tag
@@ -396,10 +394,31 @@ class DICOMwebAssetstoreAdapter(AbstractAssetstoreAdapter):
         except ValueError:
             return
 
-    def _importData(self, parent, parentType, params, progress, user):
+    def importData(self, parent, parentType, params, progress, user, **kwargs):
+        """
+        Import DICOMweb WSI instances from a DICOMweb server.
+
+        :param parent: The parent object to import into.
+        :param parentType: The model type of the parent object.
+        :type parentType: str
+        :param params: Additional parameters required for the import process.
+            This dictionary may include the following keys:
+
+            :limit: (optional) limit the number of studies imported.
+            :search_filters: (optional) a dictionary of additional search
+                filters to use with dicomweb_client's `search_for_series()`
+                function.
+
+        :type params: dict
+        :param progress: Object on which to record progress if possible.
+        :type progress: :py:class:`girder.utility.progress.ProgressContext`
+        :param user: The Girder user performing the import.
+        :type user: dict or None
+        :return: a list of items that were created
+        """
         if parentType not in ('folder', 'user', 'collection'):
             msg = f'Invalid parent type: {parentType}'
-            raise ValidationException(msg)
+            raise RuntimeError(msg)
 
         limit = params.get('limit')
         search_filters = params.get('search_filters', {})
@@ -493,65 +512,6 @@ class DICOMwebAssetstoreAdapter(AbstractAssetstoreAdapter):
 
         return items
 
-    def importData(self, parent, parentType, params, progress, user, **kwargs):
-        """
-        Import DICOMweb WSI instances from a DICOMweb server.
-
-        :param parent: The parent object to import into.
-        :param parentType: The model type of the parent object.
-        :type parentType: str
-        :param params: Additional parameters required for the import process.
-            This dictionary may include the following keys:
-
-            :limit: (optional) limit the number of studies imported.
-            :filters: (optional) a dictionary/JSON string of additional search
-                filters to use with dicomweb_client's `search_for_series()`
-                function.
-
-        :type params: dict
-        :param progress: Object on which to record progress if possible.
-        :type progress: :py:class:`girder.utility.progress.ProgressContext`
-        :param user: The Girder user performing the import.
-        :type user: dict or None
-        :return: a list of items that were created
-        """
-        # Validate the parameters
-        limit = params.get('limit') or None
-        if limit is not None:
-            error_msg = f'Invalid limit: {limit}'
-            try:
-                limit = int(limit)
-            except ValueError:
-                raise ValidationException(error_msg)
-
-            if limit < 1:
-                raise ValidationException(error_msg)
-
-        search_filters = params.get('filters', {})
-        if isinstance(search_filters, str):
-            try:
-                search_filters = json.loads(search_filters)
-            except json.JSONDecodeError as e:
-                msg = f'Invalid filters: "{params.get("filters")}". {e}'
-                raise ValidationException(msg)
-
-        items = self._importData(
-            parent,
-            parentType,
-            {
-                'limit': limit,
-                'search_filters': search_filters,
-            },
-            progress,
-            user,
-        )
-
-        if not items:
-            msg = 'No studies matching the search filters were found'
-            raise ValidationException(msg)
-
-        return items
-
     @property
     def auth_session(self):
         return _create_auth_session(self.assetstore_meta)

{large-image-source-dicom-1.28.2 → large-image-source-dicom-1.28.2.dev2}/large_image_source_dicom/assetstore/rest.py

@@ -1,14 +1,15 @@
+import json
+
 from girder.api import access
 from girder.api.describe import Description, autoDescribeRoute
 from girder.api.rest import Resource
 from girder.constants import TokenScope
 from girder.exceptions import RestException
 from girder.models.assetstore import Assetstore
+from girder.utility import assetstore_utilities
 from girder.utility.model_importer import ModelImporter
 from girder.utility.progress import ProgressContext
 
-from .dicomweb_assetstore_adapter import DICOMwebAssetstoreAdapter
-
 
 class DICOMwebAssetstoreResource(Resource):
     def __init__(self):
@@ -33,14 +34,39 @@ class DICOMwebAssetstoreResource(Resource):
         parent = ModelImporter.model(destinationType).load(params['destinationId'], force=True,
                                                            exc=True)
 
-        return DICOMwebAssetstoreAdapter(assetstore).importData(
+        limit = params.get('limit') or None
+        if limit is not None:
+            error_msg = 'Invalid limit'
+            try:
+                limit = int(limit)
+            except ValueError:
+                raise RestException(error_msg)
+
+            if limit < 1:
+                raise RestException(error_msg)
+
+        try:
+            search_filters = json.loads(params.get('filters') or '{}')
+        except json.JSONDecodeError as e:
+            msg = f'Invalid filters: {e}'
+            raise RestException(msg)
+
+        adapter = assetstore_utilities.getAssetstoreAdapter(assetstore)
+        items = adapter.importData(
             parent,
             destinationType,
-            params,
+            {
+                'limit': limit,
+                'search_filters': search_filters,
+            },
             progress,
             user,
         )
 
+        if not items:
+            msg = 'No studies matching the search filters were found'
+            raise RestException(msg)
+
     @access.admin(scope=TokenScope.DATA_WRITE)
     @autoDescribeRoute(
         Description('Import references to DICOM objects from a DICOMweb server')

{large-image-source-dicom-1.28.2 → large-image-source-dicom-1.28.2.dev2}/large_image_source_dicom/web_client/main.js

@@ -1,6 +1,7 @@
+import './routes';
+
 // Extends and overrides API
 import './constants';
-import './views/DICOMwebImportView';
 import './views/AssetstoresView';
 import './views/EditAssetstoreWidget';
 import './views/NewAssetstoreWidget';

large-image-source-dicom-1.28.2.dev2/large_image_source_dicom/web_client/models/AssetstoreModel.js

@@ -0,0 +1,20 @@
+import AssetstoreModel from '@girder/core/models/AssetstoreModel';
+import { restRequest } from '@girder/core/rest';
+
+/**
+ * Extends the core assetstore model to add DICOMweb-specific functionality.
+ */
+AssetstoreModel.prototype.dicomwebImport = function (params) {
+    return restRequest({
+        url: 'dicomweb_assetstore/' + this.get('_id') + '/import',
+        type: 'POST',
+        data: params,
+        error: null
+    }).done(() => {
+        this.trigger('g:imported');
+    }).fail((err) => {
+        this.trigger('g:error', err);
+    });
+};
+
+export default AssetstoreModel;

large-image-source-dicom-1.28.2.dev2/large_image_source_dicom/web_client/routes.js

@@ -0,0 +1,17 @@
+import router from '@girder/core/router';
+import events from '@girder/core/events';
+
+import AssetstoreModel from './models/AssetstoreModel';
+import DICOMwebImportView from './views/DICOMwebImportView';
+
+router.route('dicomweb_assetstore/:id/import', 'dwasImport', function (id) {
+    // Fetch the assetstore by id, then render the view.
+    const assetstore = new AssetstoreModel({ _id: id });
+    assetstore.once('g:fetched', function () {
+        events.trigger('g:navigateTo', DICOMwebImportView, {
+            model: assetstore
+        });
+    }).once('g:error', function () {
+        router.navigate('assetstores', { trigger: true });
+    }).fetch();
+});

{large-image-source-dicom-1.28.2 → large-image-source-dicom-1.28.2.dev2}/large_image_source_dicom/web_client/templates/dicomwebAssetstoreImportButton.pug

@@ -1,6 +1,6 @@
 .g-assetstore-import-button-container
   a.g-dwas-import-button.btn.btn-sm.btn-success(
-      href=`#assetstore/${assetstore.get('_id')}/import`,
+      href=`#dicomweb_assetstore/${assetstore.get('_id')}/import`,
       title="Import references to DICOM objects from a DICOMweb server")
     i.icon-link-ext
     |  Import data

{large-image-source-dicom-1.28.2 → large-image-source-dicom-1.28.2.dev2}/large_image_source_dicom/web_client/views/DICOMwebImportView.js

@@ -5,9 +5,6 @@ import router from '@girder/core/router';
 import View from '@girder/core/views/View';
 import { restRequest } from '@girder/core/rest';
 
-import { assetstoreImportViewMap } from '@girder/core/views/body/AssetstoresView';
-import { AssetstoreType } from '@girder/core/constants';
-
 import DWASImportTemplate from '../templates/assetstoreImport.pug';
 
 const DICOMwebImportView = View.extend({
@@ -27,12 +24,12 @@ const DICOMwebImportView = View.extend({
             }
 
             this.$('.g-submit-dwas-import').addClass('disabled');
-            this.assetstore.off().on('g:imported', function () {
+            this.model.off().on('g:imported', function () {
                 router.navigate(destinationType + '/' + destinationId, { trigger: true });
             }, this).on('g:error', function (err) {
                 this.$('.g-submit-dwas-import').removeClass('disabled');
                 this.$('.g-validation-failed-message').html(err.responseJSON.message);
-            }, this).import({
+            }, this).dicomwebImport({
                 destinationId,
                 destinationType,
                 limit,
@@ -43,7 +40,7 @@ const DICOMwebImportView = View.extend({
         'click .g-open-browser': '_openBrowser'
     },
 
-    initialize: function (settings) {
+    initialize: function () {
         this._browserWidgetView = new BrowserWidget({
             parentView: this,
             titleText: 'Destination',
@@ -78,13 +75,12 @@ const DICOMwebImportView = View.extend({
                 }
             });
         });
-        this.assetstore = settings.assetstore;
         this.render();
     },
 
     render: function () {
         this.$el.html(DWASImportTemplate({
-            assetstore: this.assetstore
+            assetstore: this.model
         }));
 
         return this;
@@ -95,6 +91,4 @@ const DICOMwebImportView = View.extend({
     }
 });
 
-assetstoreImportViewMap[AssetstoreType.DICOMWEB] = DICOMwebImportView;
-
 export default DICOMwebImportView;

{large-image-source-dicom-1.28.2 → large-image-source-dicom-1.28.2.dev2}/large_image_source_dicom.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: large-image-source-dicom
-Version: 1.28.2
+Version: 1.28.2.dev2
 Summary: A DICOM tilesource for large_image.
 Home-page: https://github.com/girder/large_image
 Author: Kitware, Inc.
@@ -17,11 +17,10 @@ Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Requires-Python: >=3.8
 License-File: LICENSE
-Requires-Dist: large-image>=1.28.2
+Requires-Dist: large-image>=1.28.2.dev2
 Requires-Dist: wsidicom>=0.9.0
 Provides-Extra: girder
-Requires-Dist: girder-large-image>=1.28.2; extra == "girder"
-Requires-Dist: girder>=3.2.3; extra == "girder"
+Requires-Dist: girder-large-image>=1.28.2.dev2; extra == "girder"
 
 A DICOM tilesource for large_image.
 

{large-image-source-dicom-1.28.2 → large-image-source-dicom-1.28.2.dev2}/large_image_source_dicom.egg-info/SOURCES.txt

@@ -20,6 +20,8 @@ large_image_source_dicom/assetstore/rest.py
 large_image_source_dicom/web_client/constants.js
 large_image_source_dicom/web_client/main.js
 large_image_source_dicom/web_client/package.json
+large_image_source_dicom/web_client/routes.js
+large_image_source_dicom/web_client/models/AssetstoreModel.js
 large_image_source_dicom/web_client/templates/assetstoreImport.pug
 large_image_source_dicom/web_client/templates/dicomwebAssetstoreCreate.pug
 large_image_source_dicom/web_client/templates/dicomwebAssetstoreEditFields.pug

large-image-source-dicom-1.28.2.dev2/large_image_source_dicom.egg-info/requires.txt

@@ -0,0 +1,5 @@
+large-image>=1.28.2.dev2
+wsidicom>=0.9.0
+
+[girder]
+girder-large-image>=1.28.2.dev2

{large-image-source-dicom-1.28.2 → large-image-source-dicom-1.28.2.dev2}/setup.py

@@ -41,14 +41,11 @@ entry_points = {
     ],
 }
 
-girder_extras = [f'girder-large-image{limit_version}']
-
 if sys.version_info >= (3, 9):
     # For Python >= 3.9, include the DICOMweb plugin
     entry_points['girder.plugin'] = [
         'dicomweb = large_image_source_dicom.girder_plugin:DICOMwebPlugin',
     ]
-    girder_extras.append('girder>=3.2.3')
 
 setup(
     name='large-image-source-dicom',
@@ -74,7 +71,7 @@ setup(
         'wsidicom>=0.9.0',
     ],
     extras_require={
-        'girder': girder_extras,
+        'girder': f'girder-large-image{limit_version}',
     },
     include_package_data=True,
     keywords='large_image, tile source',

{large-image-source-dicom-1.28.2 → large-image-source-dicom-1.28.2.dev2}/test_dicom/web_client_specs/dicomWebSpec.js

@@ -143,7 +143,7 @@ describe('DICOMWeb assetstore', function () {
         });
 
         waitsFor(function () {
-            return $('.g-validation-failed-message').html() === 'Invalid limit: 1.3';
+            return $('.g-validation-failed-message').html() === 'Invalid limit';
         }, 'Invalid limit check (float)');
 
         runs(function () {
@@ -156,7 +156,7 @@ describe('DICOMWeb assetstore', function () {
         });
 
         waitsFor(function () {
-            return $('.g-validation-failed-message').html() === 'Invalid limit: -1';
+            return $('.g-validation-failed-message').html() === 'Invalid limit';
         }, 'Invalid limit check (negative)');
 
         runs(function () {

large-image-source-dicom-1.28.2/large_image_source_dicom.egg-info/requires.txt

@@ -1,6 +0,0 @@
-large-image>=1.28.2
-wsidicom>=0.9.0
-
-[girder]
-girder-large-image>=1.28.2
-girder>=3.2.3