Fix CI tests in macro/run
All transport macros now properly depend on run_cleanup. Remove duplicate entries for json config tests. Run tests for all defined setups. With commit e6c87d7a (MR !646 (merged)) many new setups for APR21 and DEC21 were added. Unfortunately the generated file names weren't changed accordingly. This results in a situation that not all defined setups were executed. As example in the case of three setups with the names sis100_electron, sis100_electron_APR21 and sis100_electron_DEC21 only one setup was tested. This was either the first or the last defined setup. Now all defined setups are tested which increases the number of test in the run directory from 63 to 111.
Merge request reports
Activity
added BugFix Build System CI/CD labels
- Resolved by Pierre-Alain Loizeau
testing all defined setups will introduce 2 failing radiation length tests, at least I saw this locally. This is a good check to the newly integrated test.
Dear @f.uhlig, @p.-a.loizeau,
you have been identified as code owner of at least one file which was changed with this merge request.
Please check the changes and approve them or request changes.
added CodeOwners label
added 13 commits
-
d5db46cc...6076e7d8 - 11 commits from branch
computing:master
- 8376d96e - Fix CI tests in macro/run
- 818b74d7 - Fix accidentally overwritten changes
-
d5db46cc...6076e7d8 - 11 commits from branch
- Resolved by Florian Uhlig
I notice that the CI chain takes already now longer than one hour. I think we should not increase this further by adding more geometry setups to the test list.
The selection electron setup and muon setup is chosen such as to cover most of the relevant software. To check the geometries themselves (overlaps and radiation length), the full simulation and reconstruction chain is not needed; dedicated tests can of course be run.
Nightly or weekly is ok, there we test also the sis300 setups.
added 24 commits
-
171747ff...e09c1e22 - 23 commits from branch
computing:master
- 2a21eab2 - Fix CI tests in macro/run
-
171747ff...e09c1e22 - 23 commits from branch