Fluoro pulse width, kV, current and exposure can all be multi-value...
In http://dicom.nema.org/medical/dicom/current/output/chtml/part16/sect_TID_10003B.html pulse width, kV, current and exposure can all be multi-valued, ie one value of each per pulse in that event.
The database is set up for this, but the extractor routine, display and export routines are not.
I think that at least the import routine needs to be fixed asap.
Comments (17)
-
reporter -
reporter Imports without error. dxdetail displays kV as a list, rfdetail displays kV and mA as a list. Refs
#584→ <<cset 1d3aeb3ebb29>>
-
reporter DX exports working with average kVp. Not sure what is happening with exposure - need to investigate. Refs
#584→ <<cset df1c7e595f3b>>
-
reporter Adding in doc string I missed in previous commit 2588474. Refs
#584→ <<cset 50af45357039>>
-
reporter Exposure/mAs displays and exports correctly. Average/mean mAs in export, list in interface. Modified uAs to mAs conversion to return if 0. Refs
#584→ <<cset f0edc876bd7b>>
-
reporter Copied the DX export single/multiple average pulse level export to the common file. RF exports. Refs
#584→ <<cset 589dfff4709c>>
-
reporter Making use of the factored out pulse level export data. Refs
#584→ <<cset e35faff8eaa4>>
-
reporter Added quick test that the Eurocolumbus RDSR imports and exports with the correct mean values. Refs
#584→ <<cset e4b2049678c2>>
-
reporter Adding ref
#584to changes.→ <<cset 931660f42001>>
-
reporter Doc string wrong side of imports. refs
#584→ <<cset e6eaa808f342>>
-
reporter - changed status to resolved
Merged in issue584perpulsedata (pull request #151)
Issue584perpulsedata
Fixes
#584→ <<cset b56b5dbaebaf>>
-
reporter Merged in issue602multiplekVpError (pull request #162)
Approved-by: Ed McDonagh ed@mcdonagh.org.uk
→ <<cset bba534d5588a>>
-
reporter - changed status to open
-
Should be easy to do this - looks like a few exclude clauses need to be added. Do you want me to do it?
-
reporter No thanks, I'm on it. Copying your method...
-
reporter Updating values in export to be correct for mean after excluding zeros. Refs
#584→ <<cset 91a9cafa37ec>>
-
reporter - changed status to resolved
Merged in issue584muktipulsekVrevisited (pull request #166)
Fixes
#584againApproved-by: David Platten dplatten@gmail.com
→ <<cset 040ae5a7507d>>
- Log in to comment
Initial attempt to cater for per pulse data, whilst not slowing down mean pulse data imports (by far the majority). Refs
#584→ <<cset 4c2a078d4168>>