Repository navigation

Job splitter consolidate - #1126

Open

tbaudier wants to merge 13 commits into

job_splitter_consolidate

tbaudier commented Sep 3, 2026

Contributor

Serialize RepeatParametrisedVolume to be able to save it in .json the repeated volume.
We can test the splitting with actors/test036_adder_depth_param.py

tbaudier commented Sep 8, 2026

Contributor Author

For biaising actor attributes and parrallel world you can test the splitting with geometry/test054e_multi_parallel_worlds_mt.py

tbaudier commented Sep 8, 2026 •

edited

Loading

Contributor Author

For boolean_volume, can be tested with geometry/test016_bool_volumes.py
For blurring attributes in digitizer, can be tested with actors/test049_pet_digit_blurring_v3.py
For spatial blurring attributes in digitizer, can be tested with actors/test094_truncatedGaussian.py

tbaudier force-pushed the job_splitter_consolidate branch 3 times, most recently from 049dea9 to 1498d71 Compare

September 15, 2026 11:50

tbaudier commented

View reviewed changes

docs/source/user_guide/user_guide_how_to_convert_example_1.rst Outdated

tbaudier commented

View reviewed changes

opengate/jobs.py Outdated

tbaudier commented

View reviewed changes

opengate/geometry/volumes.py

+                      "repeated_volume_name": (
+                          None,
+                          {"doc": "Name of the repeated volume."},
+                      ),

tbaudier Sep 18, 2026

Contributor Author

set repeated_volume into user_info_defaults or in to_dict() function instead of repeated_volume_name

tbaudier commented

View reviewed changes

opengate/actors/biasingactors.py Outdated

tbaudier commented

View reviewed changes

opengate/actors/digitizers.py Outdated

tbaudier commented

View reviewed changes

slurm.json

tbaudier added 6 commits

October 5, 2026 17:27


          Replace .n by .number_of_primaries in the docs

8e7bab6


          Add more information in the doc and in the options for splitter to he…

aa7d5f9

…lp the user


          Save repeated_volume_name in json

0ae5b70

For RepeatParametrisedVolume, the repeated_volume was not serialized. So it was not possible to create the RepeatParametrisedVolume from a json
Now, save the repeated_volume_name and add the reference in volumes.py to be sure to have the link to the repeated_volume


          For opengate_jobs_split, set the particles attribute of biaising acto…

8a02d11

…r available to export to json files


          Be able to load parrallel worlds from json files

c80003a


          Be able to create a boolean volume from dict

bf067d4

self.volume_manager could be None so the except with ErrorKey fails to catch it
Now, we are still able to create the volume

tbaudier force-pushed the job_splitter_consolidate branch 4 times, most recently from 8399047 to 23f0035 Compare

October 6, 2026 09:16

tbaudier added 5 commits

October 7, 2026 10:20


          For Blurring actor, set only sigma from fwhm

1989f1b

Before if both were set, it returned an error
But after an export to json, both are set because previously computed
So now, we only set the sigma and if the fwhm is set by the user, with the setter and getter method it's automatically convert to sigma


          Towards a slurm running splitting method

9daafd8


          Towards pbs cluster

c7e8907


          Add doc for slurm and pbd cluster for multijobs

ec74429


          Debug linac simulation advanced test to be able to run it with multijob

9d2e1f3

Check if numpy input, to convert it to int, float, bool for the serialization

tbaudier force-pushed the job_splitter_consolidate branch from 8b5b5db to 9d2e1f3 Compare

October 7, 2026 08:35


          [pre-commit.ci] Automatic python and c++ formatting

4622e9c

tbaudier mentioned this pull request

Digitizer projection actor splitter #1153

Open


          The test test111_jobs_run_slurm.py does not work because slurm is not…

29d669d

… installed during CI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet