Clean up qrscu a little to aid field debugging

Issue #528 resolved
Ed McDonagh
created an issue

For example see if we can close each association before opening the next one

Comments (44)

  1. Ed McDonagh reporter

    See also 91a8855: Terminated initial association after ehco instead of keeping it open. Better check of initial echo. More references to query_id in logging, but not completed. Refs 528

  2. David Platten

    Hi @Ed McDonagh. Should I be able to drop the updated files into my live system to test this?

    As an aside, for my Toshiba CT RDSR creation routine to work I'll need to be able to retrieve at least two images from each series because the dose summaries consist of two images.

  3. David Platten

    @Ed McDonagh, I've used the code from this branch to query my PACS.

    python D:\Server_Apps\python27\Scripts\openrem_qr.py 1 1 -dx -f 2017-09-14 -t 2017-09-14
    

    and

    python D:\Server_Apps\python27\Scripts\openrem_qr.py 1 1 -mg -f 2017-09-14 -t 2017-09-14
    

    both worked beautifully, but when I tried

    python D:\Server_Apps\python27\Scripts\openrem_qr.py 1 1 -ct -f 2017-09-14 -t 2017-09-14 -i "CT Chest High Resolution"
    

    I received the following error:

    Traceback (most recent call last):
      File "D:\Server_Apps\python27\Scripts\openrem_qr.py", line 14, in <module>
        sys.exit(qrscu_script(sys.argv))
      File "D:\Server_Apps\python27\lib\site-packages\openrem\remapp\netdicom\qrscu.py", line 947, in qrscu_script
        study_desc_inc = map(str.lower, map(str.strip, args.desc_include.split(u',')))
    TypeError: descriptor 'strip' requires a 'str' object but received a 'unicode'
    

    Do you think this error may be because I'm running OpenREM 0.7.4 code everywhere except for your updated netdicom files?

    Thanks.

  4. Ed McDonagh reporter

    No, from the error message that is probably due to the new files, and lack of testing of that element.

    I'm glad the non-text string tests worked nicely. I'll add a test if I can and fix the versions with the text.

  5. David Platten

    @Ed McDonagh, I tried to use this new code to retrieve CT data from PACS, but it only fetches RDSR objects. Does it have a switch to make it fetch images too? I need to be able to retrieve images for the majority of our CT scanners. Thanks.

  6. David Platten

    I need to be able to get:

    • Philips dose screens from a Brilliance 64
    • Toshiba dose screens from an Aquilion CX and an Aquilion CXL. Dose screens from these come as a pair of images, so I need at least two images retrieved from each series
  7. Ed McDonagh reporter

    None of the long -- versions of the flags are working, nor are multi-letter short - flags with strings. Not sure if it's something to do with this version of the code, or if it never worked.

  8. Log in to comment