-
Notifications
You must be signed in to change notification settings - Fork 483
FST: Extend FST to allow for ITS-GPU reco #14106
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
REQUEST FOR PRODUCTION RELEASES: This will add The following labels are available |
| if [[ ! -z $INPUT_DETECTOR_LIST ]]; then | ||
| if [[ $CTFINPUT == 1 ]]; then | ||
| GPU_INPUT=compressed-clusters-ctf | ||
| has_detectors_gpu TPC ITS && GPU_INPUT+=",its-clusters" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why do you do this only for CTFINPUT=1, I think this should be global?
Simiar for the GPU_OUTPUT above. In principle, I'd set the ITS input and outputs in GPU_INPUT and GPU_OUTPUT at the same place to avoid confusion.
4213c3a to
011d361
Compare
davidrohr
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Otherwise looks good now 👍
Let me know when it is working :).
Ok, I connected also the writer, now it should be functional. Tested in PbPb with: |
|
Error while checking build/O2/fullCI_slc9 for 22e3089 at 2025-03-27 19:16: Full log here. |
Before merging:
gpu-reco-workflowinvocationRequires: AliceO2Group/O2DPG#1951