Added `Target Exposure Index` and `Deviation Index` to the radiographic exports. Fixes issue #615

#180 Merged at c9f4940
Repository
Branch
issue615addDIandTDItoDXexport
Repository
Branch
develop
Author
  1. David Platten
Reviewers
Description
No description
Code Quality

Comments (6)

  1. Ed McDonagh

    Codacy Here is an overview of what got changed by this pull request:

    Issues
    ======
    + Solved 1
    

    See the complete overview on Codacy

  2. Ed McDonagh

    Thanks David. I'm hoping that my major reorganisation of the export functions made that a not particularly painful experience?

    When you tested it, did you have any studies with more than one image in, and if so, did the headers all work correctly on the CSV and the XLSX all headers sheet? I think I may have set it to count the headers so it just works, but I can't remember.

  3. David Platten author

    It was really, really easy to add.

    I'll check what is in the output for a study with more than one image in and feedback here.

  4. David Platten author

    OK - the headers work correctly in the exports for multi-image studies in both the csv and xlsx files, insofar as there is a set of headings for each exposure and the data under these headings is correct. Is that what you meant?

  5. Ed McDonagh

    Codacy Here is an overview of what got changed by this pull request:

    Issues
    ======
    + Solved 1
    

    See the complete overview on Codacy