Duplicate study code in dx.py doesn't work

Issue #628 resolved
Ed McDonagh created an issue

In implementing the event level SOPInstanceUID for dx.py (issue #610), it has become clear that the code for dealing with multiple studies with the same study instance UID does not function.

This was partially fixed in 6cda0df, but in attempting to test that fix I have realised that the function drops out even before it gets to that code.

This aught to be fixed...

Comments (17)

  1. Ed McDonagh reporter

    Fixed image import when more than one existing study in the database matches the new image study UID. Refs #628. Added tests that helped to fix the function.

    → <<cset 06ac2c50305c>>

  2. Ed McDonagh reporter

    Explicitly named mam log for consistency. Added record_sop for new studies. Added tests for importing new event and duplicate vent a second time. Lots of duplication still... for another time! Refs #610, #628

    → <<cset 3bf064fa0fa6>>

  3. Ed McDonagh reporter

    Tested in production. check_uid.check_uid was preventing UIDs to be collected, and isn't needed in extract_common. Removed. Added debug logging to duplicates QR code. Added query stage updates for web interface. Refs #610, #628

    → <<cset 05e0457273b4>>

  4. Log in to comment