1. OpenREM
  2. OpenREM
  3. OpenREM
  4. Pull requests

Issue627ModalityBugImpax

Merged
#200 · Created  · Last updated

Merged pull request

Merged in issue627ModalityBugImpax (pull request #200)

  • 0a44ad0
  • Author:
  • Closed by:
  • 2018-08-15

 

  • Whether tag modality is used in study-request is configurable per query-node.

  • Special comment concerning the study_rsp.save() in qrscu.py:
    the .save() really seems necessary(see #626).
    Log without study_rsp.save():
    [27/Jun/2018 10:27:20] DEBUG [remapp.netdicom.qrscu:827] Pruning series responses
    [27/Jun/2018 10:27:20] INFO [remapp.netdicom.qrscu:157] Deleting series we can't use
    [27/Jun/2018 10:27:20] DEBUG [remapp.netdicom.qrscu:170] at study in study_resp in series prune. modalities in study are: [u'']
    ...
    [27/Jun/2018 10:27:21] DEBUG [remapp.netdicom.qrscu:857] Study: CT WETENSCHAP 16004 TX + ABD BMS CA209-274 I.V. CONTRAST (<STUDY_UID>) modalities: [u''], Series: <SERIES_UID>, modality: CT containing 501 objects.
    <AND MANY MORE SERIES FROM THIS STUDY>

    Log with study_rsp.save():
    [27/Jun/2018 10:29:52] DEBUG [remapp.netdicom.qrscu:827] Pruning series responses
    [27/Jun/2018 10:29:52] INFO [remapp.netdicom.qrscu:157] Deleting series we can't use
    [27/Jun/2018 10:29:52] DEBUG [remapp.netdicom.qrscu:170] at study in study_resp in series prune. modalities in study are: [u'SR', u'CT']
    ...
    [27/Jun/2018 10:30:05] DEBUG [remapp.netdicom.qrscu:857] Study: CT WETENSCHAP 16004 TX + ABD BMS CA209-274 I.V. CONTRAST (<STUDY_UID>) modalities: [u'SR', u'CT'], Series: <SERIES_UID>, modality: SR containing 1 objects.
    <THIS IS THE ONLY SERIES FROM THIS STUDY>
    So without .save() modalities in study stay empty. With .save() they are filled.

  • Updated change lists and documentation.

 

0 attachments

0 comments

Loading commits...