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

Tps no step limiter #189

Merged
merged 5 commits into from
Jul 6, 2023
Merged

Tps no step limiter #189

merged 5 commits into from
Jul 6, 2023

Conversation

tbaudier
Copy link
Contributor

Same than https://github.com/OpenGATE/opengate/pull/148/files after rebasing

created tests for tps source
Treatment plan source file was missing from the branch. Added. small modifications. Still trying to figure out the error in spot positions Fixed bug with TPS rotation and created tests
Removed old version of the helpers. Now the function that reads the treatment plan txt file returns also the gantry angle Created tests for: optics in air, absolute dose in water and gantry rotation. Working version of TPSource
Removed full MA polinomia from the beamline model
added test for weights and comparison pbs-tps
exposed G4UserLimits and StepLimiter
Corrected bugs in TPSource and created working tests for optics, absolute dose, ranges and weights Improved test metrics and TPS readability
test optics in vbl
Refactored TPS init and renamed beamline properties changed TPS initialization
minor changes, mainly estetics'
turn off visualization
reduced the size of the detectors
Further reduced the detector's side
Included energy spread in the TPS model and examples Corrected inconsistency in test for absolute dose. Added function to test dose grid shape and spacing to helpers_test accounted for correction in dose grid (PR#137)
accounted for PR#137
corrected scale_dose function for case image size 1,1,x multiprocessing: do we need a timeout in p.join()? spot weighted on the beamset number of particles, not on the beam ones. created separate neme dictionary for particles to use in user_limit Changed queue initialization for multiprocessing
added check to avoid error when beam spot has almost zero weight removed step limiter from branch
modified threshold

Martina Favaretto added 2 commits July 3, 2023 13:42
Treatment plan source file was missing from the branch. Added.
small modifications. Still trying to figure out the error in spot positions
Fixed bug with TPS rotation and created tests
Removed old version of the helpers. Now the function that reads the treatment plan txt file returns also the gantry angle
Created tests for: optics in air, absolute dose in water and gantry rotation.
Working version of TPSource
Removed full MA polinomia from the beamline model
added test for weights and comparison pbs-tps
exposed G4UserLimits and StepLimiter
Corrected bugs in TPSource and created working tests for optics, absolute dose, ranges and weights
Improved test metrics and TPS readability
test optics in vbl
Refactored TPS init and renamed beamline properties
changed TPS initialization
minor changes, mainly estetics'
turn off visualization
 reduced the size of the detectors
Further reduced the detector's side
Included energy spread in the TPS model and examples
Corrected inconsistency in test for absolute dose. Added function to test dose grid shape and spacing to helpers_test
accounted for correction in dose grid (PR#137)
accounted for PR#137
corrected scale_dose function for case image size 1,1,x
multiprocessing: do we need a timeout in p.join()?
spot weighted on the beamset number of particles, not on the beam ones.
created separate neme dictionary for particles to use in user_limit
Changed queue initialization for multiprocessing
added check to avoid error when beam spot has almost zero weight
removed step limiter from branch
modified threshold
@tbaudier tbaudier force-pushed the rebase_tps_no_step_limiter branch from f171cf2 to 549fecf Compare July 3, 2023 11:43
tbaudier and others added 3 commits July 3, 2023 17:02
…pot is a fraction of the requested particles to simulate, we round this number to have an integer. Therefore the final number of simulated particles may be different from the required one, expecially for low number of particles
@tbaudier tbaudier changed the base branch from master to uncertainty July 5, 2023 06:43
@tbaudier tbaudier changed the base branch from uncertainty to master July 5, 2023 06:44
@tbaudier
Copy link
Contributor Author

tbaudier commented Jul 6, 2023

@tbaudier tbaudier merged commit 25d3260 into master Jul 6, 2023
16 checks passed
@tbaudier tbaudier deleted the rebase_tps_no_step_limiter branch July 6, 2023 07:42
@tbaudier tbaudier mentioned this pull request Jul 6, 2023
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.

1 participant