MooreAnalysis merge requestshttps://gitlab.cern.ch/lhcb/MooreAnalysis/-/merge_requests2023-08-24T15:00:07+02:00https://gitlab.cern.ch/lhcb/MooreAnalysis/-/merge_requests/128Disable tests in dd4hep builds2023-08-24T15:00:07+02:00Christopher Rob Jonesjonesc@hep.phy.cam.ac.ukDisable tests in dd4hep buildsSee lhcb/MooreAnalysis#43See lhcb/MooreAnalysis#43https://gitlab.cern.ch/lhcb/MooreAnalysis/-/merge_requests/114Follow renaming of allen_gaudi_node in lhcb/Allen!11292023-06-26T07:38:26+02:00Roel AaijFollow renaming of allen_gaudi_node in lhcb/Allen!1129Fixes the MooreAnalysis tests for DD4HEP builds. Also, while here, removes binding of FT decoding version as this configuration option no longer does anything (see dicussion below).
Depends on lhcb/Allen!1129 and lhcb/Moore!2129Fixes the MooreAnalysis tests for DD4HEP builds. Also, while here, removes binding of FT decoding version as this configuration option no longer does anything (see dicussion below).
Depends on lhcb/Allen!1129 and lhcb/Moore!2129https://gitlab.cern.ch/lhcb/MooreAnalysis/-/merge_requests/91changes needed for new persistency with encoding keys2022-09-19T13:24:56+02:00Sevda Esenchanges needed for new persistency with encoding keysTo go with LHCb!3528 and Moore!1529To go with LHCb!3528 and Moore!1529Christopher Rob Jonesjonesc@hep.phy.cam.ac.ukSevda EsenChristopher Rob Jonesjonesc@hep.phy.cam.ac.ukhttps://gitlab.cern.ch/lhcb/MooreAnalysis/-/merge_requests/81Fixes to non-retina cluster HltEfficiencyChecker examples and other test tidy...2022-07-25T12:04:49+02:00Ross John HunterFixes to non-retina cluster HltEfficiencyChecker examples and other test tidy-upsCloses #31 and #32. Since !72, where Retina clusters became the default for Allen, the tests have all run over files with Retina clusters. However, there are still plenty of examples with no Retina clusters, which were untested and broke...Closes #31 and #32. Since !72, where Retina clusters became the default for Allen, the tests have all run over files with Retina clusters. However, there are still plenty of examples with no Retina clusters, which were untested and broken by this change. This fixes them by using the `hlt1_pp_veloSP` sequence, and adds efficiency and rate tests on the `hlt1_pp_veloSP` sequence.
### Other changes
* Removed the redundant/duplicated `hlt1_rate_example.py`, as `hlt1_rate_default.py` is exactly the same,
* Remove the `no_gec` examples with no retina clusters, since there is no such Allen sequence. These files were broken and not tested since !72,
* Remove an old TOS efficiency test: all the efficiency tests now test this functionality,
* Reveal some hidden seg faults in the tests from an erroneously caught exception in `wizard.py`,
* Then patch those seg faults: the wizard options template needs all the retina-cluster based boilerplate if using the default Allen sequence,
* Make a bit of effort to use more descriptive tuple paths, partly for clarity and partly to avoid different tests writing/reading to the same place when multithreaded,
* Try a little to improve the alignment and truncation of rates results (closes #32).
TODO:
* [x] Catch #32?
* [x] Moore MR on documentation
* [x] Kill the no-gec-no-retina-cluster examples?
* [x] Test for non-retina cluster examples
* [x] ~~Hopefully avoid hard-coding the bank types for `veloSP` sequence?~~ Postpone.
Goes with Moore!1545. Might require Allen!850 and Moore!1542 for new tests to pass (to be tested again)Sebastien PonceRoss John HunterSebastien Poncehttps://gitlab.cern.ch/lhcb/MooreAnalysis/-/merge_requests/53Disable test_hlt1_allen_no_gec.qmt2021-10-02T12:32:30+02:00Alessandro ScarabottoDisable test_hlt1_allen_no_gec.qmtDisable test that are related to Allen sequence hlt1_pp_no_gec .
See https://gitlab.cern.ch/lhcb/Allen/-/merge_requests/665 and https://gitlab.cern.ch/lhcb/Allen/-/issues/277Disable test that are related to Allen sequence hlt1_pp_no_gec .
See https://gitlab.cern.ch/lhcb/Allen/-/merge_requests/665 and https://gitlab.cern.ch/lhcb/Allen/-/issues/277Alessandro ScarabottoAlessandro Scarabottohttps://gitlab.cern.ch/lhcb/MooreAnalysis/-/merge_requests/52RefBot going live!2021-10-01T10:36:09+02:00Christoph HasseRefBot going live!https://gitlab.cern.ch/lhcb/MooreAnalysis/-/merge_requests/43Remove creative use of appendPostConfigAction by creative order of options files2021-07-22T16:41:55+02:00Sascha StahlRemove creative use of appendPostConfigAction by creative order of options filesRemove use of appendPostConfigAction in `HltIntegrationTests/tests/options/hlt2_analysis/read_hlt2.py`.
Feel free to just discard the MR.Remove use of appendPostConfigAction in `HltIntegrationTests/tests/options/hlt2_analysis/read_hlt2.py`.
Feel free to just discard the MR.https://gitlab.cern.ch/lhcb/MooreAnalysis/-/merge_requests/37Add tests for ThOr-based HLT2 lines2021-07-08T15:56:53+02:00Alex PearceAdd tests for ThOr-based HLT2 linesAdds integration tests for running HLT2 with `LHCb::Particle`-based ThOr lines and making ntuples out of the result.
Requires https://gitlab.cern.ch/gaudi/Gaudi/-/merge_requests/1210, https://gitlab.cern.ch/lhcb/Rec/-/merge_requests/242...Adds integration tests for running HLT2 with `LHCb::Particle`-based ThOr lines and making ntuples out of the result.
Requires https://gitlab.cern.ch/gaudi/Gaudi/-/merge_requests/1210, https://gitlab.cern.ch/lhcb/Rec/-/merge_requests/2425, https://gitlab.cern.ch/lhcb/Phys/-/merge_requests/933, and https://gitlab.cern.ch/lhcb/Moore/-/merge_requests/821.https://gitlab.cern.ch/lhcb/MooreAnalysis/-/merge_requests/34Add efficiency checker test that uses the sequence without a GEC in Allen2021-09-25T18:31:01+02:00Roel AaijAdd efficiency checker test that uses the sequence without a GEC in AllenUse the new feature provided by lhcb/Allen!552 and lhcb/Moore!792 to run an additional efficiency checker test using the sequence without a GEC in Allen.
Todo:
- [ ] update documentation
Depends on: lhcb/Allen!552 and lhcb/Moore!792Use the new feature provided by lhcb/Allen!552 and lhcb/Moore!792 to run an additional efficiency checker test using the sequence without a GEC in Allen.
Todo:
- [ ] update documentation
Depends on: lhcb/Allen!552 and lhcb/Moore!792https://gitlab.cern.ch/lhcb/MooreAnalysis/-/merge_requests/30Add suite of tests for HLT2 analysis2021-03-15T18:30:28+01:00Alex PearceAdd suite of tests for HLT2 analysisChecks that:
1. Moore can output a DST with Turbo candidates.
2. The DST can be read by an analysis application to produce an ntuple.
3. The ntuple contains sensible information.
Ideally these test would live in Moore, where the HLT2 l...Checks that:
1. Moore can output a DST with Turbo candidates.
2. The DST can be read by an analysis application to produce an ntuple.
3. The ntuple contains sensible information.
Ideally these test would live in Moore, where the HLT2 logic lives, but
we require access to the Analysis project for DecayTreeTuple.
The test suite lives in a new package: HltIntegrationTests.
Requires LHCb!2963, Moore!762, and Analysis!770.https://gitlab.cern.ch/lhcb/MooreAnalysis/-/merge_requests/27Fix tests for Moore!6642021-07-08T15:57:15+02:00Roel AaijFix tests for Moore!664This MR follows the configuration changes in Moore!664
In addition ROOT is set to batch mode in the scripts that create canvases.
Closes #15This MR follows the configuration changes in Moore!664
In addition ROOT is set to batch mode in the scripts that create canvases.
Closes #15Carla Gobel Burlamaqui De MelloCarla Gobel Burlamaqui De Mellohttps://gitlab.cern.ch/lhcb/MooreAnalysis/-/merge_requests/24Increase timeouts for tests that have timed out in the last week2021-07-08T15:58:17+02:00Ross John HunterIncrease timeouts for tests that have timed out in the last weekI looked through the nightly build logs for the last week and have increased the timeouts for tests that failed over that period. Note that the timeout for `test_hlt2_example.py` was already increased in !22. Might need a run in the nigh...I looked through the nightly build logs for the last week and have increased the timeouts for tests that failed over that period. Note that the timeout for `test_hlt2_example.py` was already increased in !22. Might need a run in the nightlies to make sure that there are no timeouts there before merging.
Closes #14.Miroslav Saurmiroslav.saur@cern.chMiroslav Saurmiroslav.saur@cern.ch