DaVinci merge requestshttps://gitlab.cern.ch/lhcb/DaVinci/-/merge_requests2023-11-09T17:19:39+01:00https://gitlab.cern.ch/lhcb/DaVinci/-/merge_requests/994update test refs, 2018-patches2023-11-09T17:19:39+01:00Mark Smithupdate test refs, 2018-patchesFYI @avenkate @ngrieserFYI @avenkate @ngrieserhttps://gitlab.cern.ch/lhcb/DaVinci/-/merge_requests/904notify user if a specified option is overruled, fix python test2023-11-03T10:12:44+01:00Gerhard Ravennotify user if a specified option is overruled, fix python test1. notify user if the specified 'input_stream' is ignored - and do not specify a stream in one of the tests that ends up being ignored.
2. fix the python test which runs multiple configurations inside a single process by resetting the gl...1. notify user if the specified 'input_stream' is ignored - and do not specify a stream in one of the tests that ends up being ignored.
2. fix the python test which runs multiple configurations inside a single process by resetting the global 'identity' registry before each test
Goes with LHCb!4116https://gitlab.cern.ch/lhcb/DaVinci/-/merge_requests/992Resolve "Add a useful error to users when running over xdigi files"2023-11-01T10:29:21+01:00Davide FazziniResolve "Add a useful error to users when running over xdigi files"Closes #152Closes #152Davide FazziniDavide Fazzinihttps://gitlab.cern.ch/lhcb/DaVinci/-/merge_requests/989Fixed import statement2023-10-31T11:38:55+01:00Max WiegertjesFixed import statementThe given module davinci.reco_ocjects does not exist. Changed it to PyConf.reading, as is used in other examples.The given module davinci.reco_ocjects does not exist. Changed it to PyConf.reading, as is used in other examples.Abhijit MathadAbhijit Mathadhttps://gitlab.cern.ch/lhcb/DaVinci/-/merge_requests/971Implement consistency checks in configuration of jobs on Spruced data2023-10-30T11:55:52+01:00Eduardo RodriguesImplement consistency checks in configuration of jobs on Spruced dataAs per our discussions @nskidmor and @pkoppenb.
This partially addresses issue https://gitlab.cern.ch/lhcb/DaVinci/-/issues/143, @cburr, as far as DaVinci jobs are concerned. Of course one should still address separately (guess in Gaudi...As per our discussions @nskidmor and @pkoppenb.
This partially addresses issue https://gitlab.cern.ch/lhcb/DaVinci/-/issues/143, @cburr, as far as DaVinci jobs are concerned. Of course one should still address separately (guess in Gaudi) the fact that EventSelector is not safe enough.Eduardo RodriguesPatrick KoppenburgEduardo Rodrigueshttps://gitlab.cern.ch/lhcb/DaVinci/-/merge_requests/987Update xgen input file location2023-10-27T13:49:50+02:00Davide FazziniUpdate xgen input file locationUpdate the test input file locations from `/eos/lhcb/wg/dpa/wp3/testDV_read_xgen/` and `/eos/lhcb/wg/dpa/wp3/NovemberFEST/` into `/eos/lhcb/wg/dpa/wp3/tests/`.
Relates to #95.Update the test input file locations from `/eos/lhcb/wg/dpa/wp3/testDV_read_xgen/` and `/eos/lhcb/wg/dpa/wp3/NovemberFEST/` into `/eos/lhcb/wg/dpa/wp3/tests/`.
Relates to #95.Davide FazziniDavide Fazzinihttps://gitlab.cern.ch/lhcb/DaVinci/-/merge_requests/985Fix example for spruced data2023-10-27T13:49:50+02:00Davide FazziniFix example for spruced dataRemove unused option file: `option_davinci_tupling_from_data.py` and `example_data/Spruce_Run251342.yaml` in DaVinciExamples.
The file `option_davinci_tupling_from_data.py` was a copy of `option_davinci_tupling_from_raw_data.py` running ...Remove unused option file: `option_davinci_tupling_from_data.py` and `example_data/Spruce_Run251342.yaml` in DaVinciExamples.
The file `option_davinci_tupling_from_data.py` was a copy of `option_davinci_tupling_from_raw_data.py` running over a spruced line instead of a Hlt2 line. However the related input file was simulation and not real data.
Replacing the codes with a more updated working and tested example.
Fixing also the doc string of the [option_davinci_tupling_from_raw_data.py](https://gitlab.cern.ch/lhcb/DaVinci/-/blob/master/DaVinciExamples/python/DaVinciExamples/tupling/option_davinci_tupling_from_raw_data.py) script, reporting the usage of the `Spruce_Run251342.yaml` option file while using the `Run255620.yaml` script in the related qmtest [test_davinci_tupling_from_raw_data.qmt](https://gitlab.cern.ch/lhcb/DaVinci/-/blob/master/DaVinciExamples/tests/qmtest/tupling.qms/test_davinci_tupling_from_raw_data.qmt).
Closes #139.Davide FazziniDavide Fazzinihttps://gitlab.cern.ch/lhcb/DaVinci/-/merge_requests/799follow changes in Rec!3231 - replace a LoKi functor with ThOr2023-10-24T08:00:10+02:00Gerhard Ravenfollow changes in Rec!3231 - replace a LoKi functor with ThOrrequired for Rec!3231required for Rec!3231https://gitlab.cern.ch/lhcb/DaVinci/-/merge_requests/982Release notes for v44r11p62023-10-16T21:07:22+02:00Mark SmithRelease notes for v44r11p6https://gitlab.cern.ch/lhcb/DaVinci/-/merge_requests/981DV Tutorials and Examples pages now have a link to "Running DaVinci" section2023-10-13T15:44:26+02:00Maurizio MartinelliDV Tutorials and Examples pages now have a link to "Running DaVinci" sectionFollowing issue https://gitlab.cern.ch/lhcb/DaVinci/-/issues/150, the automatic documentation is modified to link "Running DaVinci" section.
In that section now also opening a `bash` shell with DaVinci environment is documented.Following issue https://gitlab.cern.ch/lhcb/DaVinci/-/issues/150, the automatic documentation is modified to link "Running DaVinci" section.
In that section now also opening a `bash` shell with DaVinci environment is documented.Maurizio MartinelliMaurizio Martinellihttps://gitlab.cern.ch/lhcb/DaVinci/-/merge_requests/980Release v63r142023-10-13T14:55:59+02:00Davide FazziniRelease v63r14Davide FazziniDavide Fazzinihttps://gitlab.cern.ch/lhcb/DaVinci/-/merge_requests/966Fix errors caused by DD4Hep future upgrades refactor2023-10-13T11:29:54+02:00Emir MuhammadFix errors caused by DD4Hep future upgrades refactorThis merge request fixes the errors caused by Detector!428. This is mostly done by replacing all instances of `trunk` with `run3/trunk` and equivalent.
This merge request is part of the larger 'detector refactoring for future simulatio...This merge request fixes the errors caused by Detector!428. This is mostly done by replacing all instances of `trunk` with `run3/trunk` and equivalent.
This merge request is part of the larger 'detector refactoring for future simulation' task, and these set of merge requests are expected to come in together. See lhcb/LHCb!4283 for which merge requests come together in the stack.
# DaVinci specific notes
This may require a reference update, as I have purposely left any file with `.ref` alone.
# TODO
- [x] Add Patrick's suggestion to LHCb
- [x] Check for any more `trunk` -> `run3/trunk`
- [x] Understand why the CI test failsRosen MatevMenglin Xumenglin.xu@cern.chEmir MuhammadRosen Matevhttps://gitlab.cern.ch/lhcb/DaVinci/-/merge_requests/876Example to store hlt2 tistos2023-10-13T11:29:14+02:00Sascha StahlExample to store hlt2 tistosDV example to store tistos information
To be tested with: https://gitlab.cern.ch/lhcb/Analysis/-/merge_requests/997 and https://gitlab.cern.ch/lhcb/Rec/-/merge_requests/3363DV example to store tistos information
To be tested with: https://gitlab.cern.ch/lhcb/Analysis/-/merge_requests/997 and https://gitlab.cern.ch/lhcb/Rec/-/merge_requests/3363Rosen MatevPatrick KoppenburgRosen Matevhttps://gitlab.cern.ch/lhcb/DaVinci/-/merge_requests/978Fix the integration test on tuple from turbo with veloSP2023-10-05T14:05:56+02:00Davide FazziniFix the integration test on tuple from turbo with veloSPFix the integration test "Tupling_veloSP.TupleFromTurbo". Failure reported in LHCbIntegrationTests#5.
Closes LHCbIntegrationTests#5.Fix the integration test "Tupling_veloSP.TupleFromTurbo". Failure reported in LHCbIntegrationTests#5.
Closes LHCbIntegrationTests#5.Davide FazziniDavide Fazzinihttps://gitlab.cern.ch/lhcb/DaVinci/-/merge_requests/979Remove evt_pre_filters from the DaVinci options2023-10-05T14:04:34+02:00Davide FazziniRemove evt_pre_filters from the DaVinci optionsRemove the `evt_pre_filters` option since it is not compatible with the ThOr functors, as reported in #115.
Closes https://gitlab.cern.ch/lhcb/DaVinci/-/issues/115.Remove the `evt_pre_filters` option since it is not compatible with the ThOr functors, as reported in #115.
Closes https://gitlab.cern.ch/lhcb/DaVinci/-/issues/115.Abhijit MathadDavide FazziniAbhijit Mathadhttps://gitlab.cern.ch/lhcb/DaVinci/-/merge_requests/977pre-commit updates2023-10-03T10:43:27+02:00Eduardo Rodriguespre-commit updateshttps://gitlab.cern.ch/lhcb/DaVinci/-/merge_requests/976Fix docs yet again while updating some CI dependencies2023-09-29T14:56:31+02:00Eduardo RodriguesFix docs yet again while updating some CI dependenciesCloses https://gitlab.cern.ch/lhcb/DaVinci/-/issues/148.Closes https://gitlab.cern.ch/lhcb/DaVinci/-/issues/148.Eduardo RodriguesEduardo Rodrigueshttps://gitlab.cern.ch/lhcb/DaVinci/-/merge_requests/975Fix documentation in tupling_relation_isovariables2023-09-28T14:24:07+02:00Tommaso Fulghesutommaso.fulghesu@cern.chFix documentation in tupling_relation_isovariablesFollowing [this thread](https://mattermost.web.cern.ch/lhcb/pl/3d8n7exo4tr4imcfbh76gma9bo ) in DPA WP3 mattermost channel (thanks @lugrazet for spotting the issue), hot fix to the documentation to avoid further similar issues in the future!Following [this thread](https://mattermost.web.cern.ch/lhcb/pl/3d8n7exo4tr4imcfbh76gma9bo ) in DPA WP3 mattermost channel (thanks @lugrazet for spotting the issue), hot fix to the documentation to avoid further similar issues in the future!Tommaso Fulghesutommaso.fulghesu@cern.chTommaso Fulghesutommaso.fulghesu@cern.chhttps://gitlab.cern.ch/lhcb/DaVinci/-/merge_requests/973Add Clang formatting as a pre-commit hook following Gaudi2023-09-28T14:19:28+02:00Eduardo RodriguesAdd Clang formatting as a pre-commit hook following GaudiAs MR https://gitlab.cern.ch/lhcb/Analysis/-/merge_requests/1013 in Analysis.As MR https://gitlab.cern.ch/lhcb/Analysis/-/merge_requests/1013 in Analysis.Eduardo RodriguesEduardo Rodrigueshttps://gitlab.cern.ch/lhcb/DaVinci/-/merge_requests/953neutral proto particle integration test options2023-09-27T23:40:13+02:00Sevda Esenneutral proto particle integration test optionsAdd an option file to go with integration test in https://gitlab.cern.ch/lhcb/LHCbIntegrationTests/-/merge_requests/48Add an option file to go with integration test in https://gitlab.cern.ch/lhcb/LHCbIntegrationTests/-/merge_requests/48Patrick KoppenburgTara Nanut PetricPatrick Koppenburg