Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change DVH.CSV export: Change dvh output to percentvolume vs dose #292

Conversation

AnnieAnNguyen
Copy link
Contributor

The current DVH.CSV export has columns in 'centiGray' and rows in '%'.

Change output to columns in '%' (100%, 99.5%, 99.0%, .....) and rows in 'centiGray' (no decimals, normal rounding). This will produce a spreadsheet with a fixed number of columns irrespective of the prescribed radiation dose.

Copy link
Collaborator

@sjswerdloff sjswerdloff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.
You might want to add some comments regarding the meaning of some of the algorithm and variables (peak, trough), etc. but I see that there is some interpolation going on, which was at the heart of our algorithm discussion.

@sjswerdloff
Copy link
Collaborator

The new CSV files contain what I would refer to as Volume Dose Histograms, rather than Dose Volume Histograms (given that the axes have essentially been swapped).

@sjswerdloff
Copy link
Collaborator

image

@sjswerdloff sjswerdloff merged commit 229ad11 into didymo:master Aug 31, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants