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

Possible to make gamma_shell operate using purely relative dose? #611

Closed
BananaOrbit opened this issue Dec 15, 2019 · 6 comments · Fixed by #613 or #626
Closed

Possible to make gamma_shell operate using purely relative dose? #611

BananaOrbit opened this issue Dec 15, 2019 · 6 comments · Fixed by #613 or #626

Comments

@BananaOrbit
Copy link

I have two PDDs I want to compare using gamma_shell in 1D mode. Both of the numpy arrays are in units of % of D_max (numbers from 0 to 1). When I run gamma_shell, it assumes that the maximum dose is 1 Gy rather than unitless number 1 or 100%.

Calcing using global normalisation point for gamma
Global normalisation set to 1.0 Gy
Global dose threshold set to [0.02] Gy ([2.]%)
Distance threshold set to [2.] mm
Lower dose cutoff set to 0.1 Gy (10%)

Could we include an option for the dose units on both the reference and evaluation dose to be in units of % or in unitless values in the interval [0,1]?

@SimonBiggs
Copy link
Member

I could just remove the reference to Gy in the output?

@BananaOrbit
Copy link
Author

The analysis works fine as is, I just think it may be a good idea for the user to specify what units the dose arrays are in (Gy, cGy, %, etc...) and the output be adjusted according to that.

@SimonBiggs
Copy link
Member

My preference is to have no input into the function be required that isn't absolutely necessary. I'll remove the units from the output given that the input arrays are unitless.

@SimonBiggs
Copy link
Member

@BananaOrbit can you verify that this is fixed in https://github.com/pymedphys/pymedphys/releases/tag/v0.14.1 by installing 0.14.1

Cheers,
Simon

@BananaOrbit
Copy link
Author

@BananaOrbit can you verify that this is fixed

Seems to be fixed! Thanks for that :)
Although there seem to be double percent signs being written in the output:

Calcing using global normalisation point for gamma
Global normalisation set to 1.0
Global dose threshold set to [0.02] ([2.]%% of normalisation)
Distance threshold set to [2.]
Lower dose cutoff set to 0.1 (10%% of normalisation)

@SimonBiggs
Copy link
Member

SimonBiggs commented Dec 16, 2019 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants