Closed dsarrut closed 5 months ago
Sure, we can discuss that. What don't you like about it?
On Feb 29 2024, at 8:53 pm, David Sarrut @.***> wrote:
I adapted tolerances + set seeds + added execution rights to some tests.
@MartiDvrk (https://github.com/MartiDvrk), @andiresch (https://github.com/andiresch) it seems that the test test059_tpsource_weights.py fails. Can you please check why?
thanks !
@nkrah (https://github.com/nkrah) unsure to like the default dump for the label_to_mat file. To be discussed
You can view, comment on, or merge this pull request online at:
https://github.com/OpenGATE/opengate/pull/362
Commit Summary
0de1052 (https://github.com/OpenGATE/opengate/pull/362/commits/0de105270491d8d05c9859abcf5b09c62decd2f5) adapt tolerance and set seed
9a85660 (https://github.com/OpenGATE/opengate/pull/362/commits/9a8566002aecae57f73f593146e88187b4af3861) do not write file in current folder
c06050a (https://github.com/OpenGATE/opengate/pull/362/commits/c06050a467424c6b3c0ee14b13a495b41bff504a) label_to_mat default json filename
07618a3 (https://github.com/OpenGATE/opengate/pull/362/commits/07618a3ddac5543c6e30621f05eac382f98c8dcc) output folder
File Changes
(24 files (https://github.com/OpenGATE/opengate/pull/362/files))
M opengate/geometry/volumes.py (https://github.com/OpenGATE/opengate/pull/362/files#diff-4677f4cb3895e4a537180027ccf7b03634244029d6e5d47c3e628590b11c5b60) (17)
M opengate/tests/src/test009_voxels.py (https://github.com/OpenGATE/opengate/pull/362/files#diff-83361dfa5d5870903aa74e1b712c6db8b3b7bab088cfbda33b96f3c9ed287523) (2)
M opengate/tests/src/test009_voxels_dynamic.py (https://github.com/OpenGATE/opengate/pull/362/files#diff-fab4e7d649e3a5b0798084f0488ed059575129ecb79598063e1677b74d21c0f8) (2)
M opengate/tests/src/test009_voxels_hu.py (https://github.com/OpenGATE/opengate/pull/362/files#diff-1c989deebf99599e481a07c493828b564282753f9a7f62c755e060c6c15f681c) (2)
M opengate/tests/src/test013_phys_lists_6.py (https://github.com/OpenGATE/opengate/pull/362/files#diff-e5be51f8ece3f9feb9f000e2864e07285b2eafccb2d375cf082641f04cfea3f1) (0)
M opengate/tests/src/test020_profiling.py (https://github.com/OpenGATE/opengate/pull/362/files#diff-94390f1402665c19af01688f9fa82cce07d569adec89fc8c45c9da10e9b454e3) (2)
M opengate/tests/src/test022_half_life_helpers.py (https://github.com/OpenGATE/opengate/pull/362/files#diff-0c39b84331cce7da77898638d2829acb26c379543b1d227dae4115794cb4637e) (0)
M opengate/tests/src/test041_dose_actor_dose_to_water_mt.py (https://github.com/OpenGATE/opengate/pull/362/files#diff-f0a30a72419c2750d908b058990b3c32a1215fedbe4e33ff104a9e3b0944b2e0) (0)
M opengate/tests/src/test041_dose_actor_mt_cp_images_Nthreads.py (https://github.com/OpenGATE/opengate/pull/362/files#diff-33debb3b04ccb8ff822661cae9398a416ef0ff7c7e77f9d5f342af4b0fb2dc88) (0)
M opengate/tests/src/test041_dose_actor_mt_standard_error_of_mean.py (https://github.com/OpenGATE/opengate/pull/362/files#diff-01970de23c3ee23847942d96af091e8f20b4740bfa9a6df64c8bc26ed093217d) (0)
M opengate/tests/src/test045_speedup.py (https://github.com/OpenGATE/opengate/pull/362/files#diff-f2a3a449c043eabf380d3530c7ab35fe1347e8f253dd1ad628829cb059694559) (12)
M opengate/tests/src/test058_uncertainty_flags_over_multiple_runs_mt.py (https://github.com/OpenGATE/opengate/pull/362/files#diff-c9f551a82978717319b1e32f89ef34cec0793ffa32b0c3f26c4fd48cbbf5f0e6) (0)
M opengate/tests/src/test059_tpsource_abs_dose.py (https://github.com/OpenGATE/opengate/pull/362/files#diff-638843b5d816187c7d108dfc3b2ad9aee284d9a89b46826fc5f28bfcff8bb15f) (5)
M opengate/tests/src/test059_tpsource_flat_generation_flag.py (https://github.com/OpenGATE/opengate/pull/362/files#diff-d0d68e2608555661dadf03f469c642888289809060fd732fcc31b61b77170c96) (0)
M opengate/tests/src/test060_phsp_source_untilNextPrimary.py (https://github.com/OpenGATE/opengate/pull/362/files#diff-64154821075ed539c150b179c0e784ec11945a28945a30d309c7b79ebb45f04e) (0)
M opengate/tests/src/test061_TPPhsSource.py (https://github.com/OpenGATE/opengate/pull/362/files#diff-0ef73ece01c9f29a4febd76690883c4743c09b1b76f457c821c994dd5f6197b2) (0)
M opengate/tests/src/test065_dose2water_from_edep2water_ct.py (https://github.com/OpenGATE/opengate/pull/362/files#diff-7e883c0f7989e839a083063f5be3153cbc8da01e1ab1913c901943dece0b02f5) (0)
M opengate/tests/src/test065_dose_from_edep_ct.py (https://github.com/OpenGATE/opengate/pull/362/files#diff-b00b879ecf5f5b8fb4822fcbe90071aaf17c4d4ccc985d28f628bed298a40264) (0)
M opengate/tests/src/test065_dose_from_edep_volume.py (https://github.com/OpenGATE/opengate/pull/362/files#diff-06d6ae09359cc8405ab8d0f4f6c6aa580ab7f7c34fa6e9593cb5716f7027b66d) (0)
M opengate/tests/src/test066_phspsource_activity.py (https://github.com/OpenGATE/opengate/pull/362/files#diff-ae7158065c71c31c3bb594b2f3fe8ed984658b80045ac40eb3064c7b3b99bc99) (0)
M opengate/tests/src/test066_stop_simulation_criteria_mt.py (https://github.com/OpenGATE/opengate/pull/362/files#diff-52cc2ba5c2ccb378a81a959ccec9ab77d2474ba82aff0658b57f45eac24e5bf6) (9)
M opengate/tests/src/test067_cbct_fluence_actor_mt.py (https://github.com/OpenGATE/opengate/pull/362/files#diff-7ccaf8f1cb4686a189c55693e7cb23c1d26a86bbfaace01d7d680f2ee66a7bd9) (15)
M opengate/tests/src/test068_rotation_source.py (https://github.com/OpenGATE/opengate/pull/362/files#diff-fb9daef98320ad218f29c630c5a68b4ebbac184528cef93209a04cd89666462d) (21)
M opengate/tests/src/test069_rotation_DICOM_RT_plan.py (https://github.com/OpenGATE/opengate/pull/362/files#diff-79b79c28041a0ad397fae5b2d188e0f7fdd6ce60b56ad94c4db54c0bae1a1ed5) (4)
Patch Links:
https://github.com/OpenGATE/opengate/pull/362.patch
https://github.com/OpenGATE/opengate/pull/362.diff
—
Reply to this email directly, view it on GitHub (https://github.com/OpenGATE/opengate/pull/362), or unsubscribe (https://github.com/notifications/unsubscribe-auth/AIFQFYPPQUAPMRIOOM6PZPLYV6DLDAVCNFSM6AAAAABEARJ7GGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGE3DCOJXGYZDCMA).
You are receiving this because you were mentioned.
something wrong here, a lot of tests fail in the mac-os-3.10 while all tests pass with other architectures.
KeyError: "Error in the Opengate library (C++): Cannot find the key 'gray' in the list of keys: _name element_type type_name mother filters filters_boolean_operator priority size spacing output translation img_coord_system output_origin uncertainty square physical_volume_index hit_type dose to_water use_more_ram ste_of_mean ste_of_mean_unbiased goal_uncertainty thresh_voxel_edep_for_unc_calc dose_calc_on_the_fly "
Strange. At want point is this error provoked?
On Mar 1 2024, at 1:59 pm, David Sarrut @.***> wrote:
something wrong here, a lot of tests fail in the mac-os-3.10 while all tests pass with other architectures.
`KeyError: "Error in the Opengate library (C++): Cannot find the key 'gray' in the list of keys: _name element_type type_name mother filters filters_boolean_operator priority size spacing output translation img_coord_system output_origin uncertainty square physical_volume_index hit_type dose to_water use_more_ram ste_of_mean ste_of_mean_unbiased goal_uncertainty thresh_voxel_edep_for_unc_calc dose_calc_on_the_fly "
—
Reply to this email directly, view it on GitHub (https://github.com/OpenGATE/opengate/pull/362#issuecomment-1973153714), or unsubscribe (https://github.com/notifications/unsubscribe-auth/AIFQFYMQF46MUU4APMAJCF3YWB3SJAVCNFSM6AAAAABEARJ7GGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZTGE2TGNZRGQ).
You are receiving this because you were mentioned.
Almost ok, but one test is not working on my computer: test059_tpsource_weights.py Can you please look at it @MartiDvrk or @andiresch thanks ;) !
Check : test069_rotation_DICOM_RT_plan_dynamic.py
Rename opengate_tests_utils
to opengate_library_path
Almost ok, but one test is not working on my computer: test059_tpsource_weights.py Can you please look at it @MartiDvrk or @andiresch thanks ;) !
as mentioned in the big PR we just closed, for me the test never fails, also after removing the output folder. The only thing that could be different is the plan file. Can you try to use this one? Maybe we changed it at some point and I forgot to update it in the data. If that doesn't solve the issue, I will further investigate. TreatmentPlan2Spots.txt
Almost ok, but one test is not working on my computer: test059_tpsource_weights.py Can you please look at it @MartiDvrk or @andiresch thanks ;) !
as mentioned in the big PR we just closed, for me the test never fails, also after removing the output folder. The only thing that could be different is the plan file. Can you try to use this one? Maybe we changed it at some point and I forgot to update it in the data. If that doesn't solve the issue, I will further investigate. TreatmentPlan2Spots.txt
Hello, I don't know if this is solved, but you can check that the rotation axis for the tp source is 'x' and the rotation is 90 deg.
yes, it is ok now, thanks @MartiDvrk, I just changed the TreatmanPlan2Spots.txt you send ;)
I adapted tolerances + set seeds + added execution rights to some tests.
@MartiDvrk, @andiresch it seems that the test test059_tpsource_weights.py fails. Can you please check why? thanks !
@nkrah unsure to like the default dump for the label_to_mat file. To be discussed