1. OpenREM
  2. OpenREM
  3. OpenREM
  4. Issues

Issues

Issue #122 resolved

Improve fluoroscopy study export

Ed McDonagh
created an issue

Current fluoro export is very high level summary to single sheet csv file.

Need to come up with an export that is useful to allow analysis, taking into account the number of exposures can be in the hundreds.

Comments (24)

  1. Ed McDonagh reporter

    Added in the code for getting the min, mean and max DAP, kVp, mA and time. Need to pre-filter by protocol and field size and loop this all so that it can iterate through the original queryset. Refs #122.

    → <<cset 1f49224b68aa>>

  2. Ed McDonagh reporter

    Previous version of similarexposures was multiplying each time an additional filter condition was added. Now the study_UID of the exam is obtained and the exam is opened at the Irradiation_event_xray_data table, which makes the rest of the code easier to read and now the filtering works. Refs #122.

    → <<cset b6cb4a00e9ad>>

  3. Ed McDonagh reporter

    Found the errant missing bracket. Added import of aggregate functions. Replaced two references to DX. Replaced numrows with e.count() as the original declaration has now moved down the routine. Added the field size to the headers. Export now basically works for the all data sheet, series sheets are empty as expected and summary sheet doesn't work with respect to the series either. Refs #122.

    → <<cset 0742df180dae>>

  4. Ed McDonagh reporter

    Made all filters check if present except the first one. Switched the exclude to exlude the similarexposures rather than trying to filter again. Currently failing due to filter_material not always existing - need to add try. Refs #122.

    → <<cset 26abc349a5ed>>

  5. Ed McDonagh reporter

    Values for filtering with are now obtained using get_db_value which doens't crash if the value isn't there as it uses the getattr with a try statement. Calls are nested with .get() as appropriate as getattr can't follow foreignkey lookups. Refs #122

    → <<cset 92c6d8570049>>

  6. Ed McDonagh reporter

    Moved the creation of tabs till after the all-data sheet has been populated. Created new code to start populating the individual dose sheets. Currently has some print statements in; it is working for the first protocol but doesn't get to the second one. Refs #122

    → <<cset 7006c1b270da>>

  7. Ed McDonagh reporter

    Now have a working export again. Variable commonheaders removed as can call _rf_common_headers each time. Now events are filtered from scratch at the irradiation_event_xray_data level, both for the protocol in question and for the list of studies in the original export filter. Needs the list of exported variables to be expanded. Refs #122.

    → <<cset 6e4f4f59a250>>

  8. Log in to comment