Mammo tube current recorded incorrectly

[Ed McDonagh]

Mam.py incorrectly puts the XrayTubeCurrent into the average_x-ray_tube_current field, and doesn't populate the x-ray_tube_current field in the tube current table.

Rdrs.py does populate the fields correctly.

Therefore when display and export look to the average field, the RDSR imported mammo doesn't have any content.

[Ed McDonagh]

Credit to Erik-Jan Rijkhorst for bringing this to my attention. Relates to 0.7b13

[Ed McDonagh]

The tube current in a GE mammo image, when multiplied with the exposure time, does not add up to the 'Exposure' or the 'Exposure in μAs'. This might suggest that the 'pre-exposure' is included in the Exposure values but the mA and time only refer to the main exposure?

[Ed McDonagh]

The same is true for Hologic 2D AEC exposures. So I think we know about one tube current event within an exposure, but we don't know the average tube current for the exposure as we've no record of the relative contributions of time and current to the difference in recorded mAs.

Propose:

• put the tube current recorded in the RDSR into the tube current field, then for mammo only populate the average tube current field, if it isn't populated, by the average of the tube currents for that exposure. Which for all systems I've seen so far, would be the single value entered. The existing display and export of average mA would continue to work.
• Start populating the xray_tube_current field in addition to the average for mammo images.

[Ed McDonagh]

Changed the pulse_width, kVp, tube_current and X-ray filters recording so that multiple values can be recorded, as had been intended. Previously only the final value in any irradiation event would be recorded. Also changed all but first if to elif to reduce computation time. Refs #371

→ <<cset d0ebec1ee935>>

[Ed McDonagh]

Attempt to create average tube current for an event from all the tube currents listed for that event. Currently not restricted to mammo. Refs #371

→ <<cset cff791d5f28b>>

[Ed McDonagh]

Forgot to save table, and result is dict. Tested with Hologic RDSR, works. Refs #371

→ <<cset c0e5090a800e>>

[Ed McDonagh]

Added test for mammo - don't want to make this assumption for fluoro. Refs #371

→ <<cset 33e715c51f1d>>

[Ed McDonagh]

Added population of xraytubecurrent from mammo image extraction. Fixes #371

→ <<cset 761e78f276bf>>

[Ed McDonagh]

Added ref #371 to changes

→ <<cset 4bc2c970b254>>

[Ed McDonagh]

Realised that the fix for average mA had previously been implemented for mammo RDSR! Refs #371, #464

→ <<cset a9bc1632380a>>