search

commontk / CTK

A set of common support code for medical imaging, surgical navigation, and related purposes.
https://commontk.org
Apache License 2.0
827 stars 480 forks source link

ENH: Add remove signal to DICOM browser #1164

Closed cpoetteCertis closed 5 months ago

cpoetteCertis commented 5 months ago

In order to fix slicer's bug described in the following link:

https://github.com/Slicer/Slicer/issues/7290

It is necessary to emit a signal when removing series from DICOM database

Punzo commented 5 months ago

@cpoetteCertis thanks for the contributions. Adding the signals makes sense. @jcfr it could make sense to add this commit https://github.com/commontk/CTK/pull/1164/commits/013d9a4f3c5dc2b389c2412460c9cf0bbb1d15fe to https://github.com/commontk/CTK/pull/1165 as well

jcfr commented 5 months ago

Adding the signals makes sense.

The commit can indeed be cherry-picked cleanly and it has been integrated.

jcfr commented 5 months ago

Closing. This is superseded by https://github.com/commontk/CTK/pull/1183