Switch table producers that produce V0s to produce V0s_001 temporarily#4156
Merged
njacazio merged 2 commits intoAliceO2Group:masterfrom Dec 13, 2023
Merged
Switch table producers that produce V0s to produce V0s_001 temporarily#4156njacazio merged 2 commits intoAliceO2Group:masterfrom
njacazio merged 2 commits intoAliceO2Group:masterfrom
Conversation
ddobrigk
requested review from
BongHwi,
ChiaraDeMartin95,
alcaliva,
alibuild,
chiarapinto,
iravasen,
lbariogl,
lbarnby,
lramona,
mbombara,
njacazio and
skundu692
as code owners
njacazio
approved these changes
Dec 13, 2023
Collaborator
njacazio
left a comment
njacazio
left a comment
There was a problem hiding this comment.
Thanks for following this David
chiarazampolli
pushed a commit
that referenced
this pull request
Dec 13, 2023
#4156) * lambdakzerofinder.cxx -> temporarily produce V0s_001 * lambdakzeromcfinder.cxx -> temporarily produce V0s_001
hahassan7
pushed a commit
to hahassan7/O2Physics
that referenced
this pull request
Feb 1, 2024
AliceO2Group#4156) * lambdakzerofinder.cxx -> temporarily produce V0s_001 * lambdakzeromcfinder.cxx -> temporarily produce V0s_001
ercolessi
pushed a commit
to ercolessi/O2Physics
that referenced
this pull request
Feb 28, 2024
AliceO2Group#4156) * lambdakzerofinder.cxx -> temporarily produce V0s_001 * lambdakzeromcfinder.cxx -> temporarily produce V0s_001
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
@shahor02 this is required so that the change in O2 can go or, indeed, the analysis CI might complain as the new version of O2 will not compile against the older version of O2Physics. Once O2Physics switches to being built against the new O2, I can switch these tasks to produce the generic
V0s.Technically, once this PR is merged, we could ignore the fullCI because then for sure the compilation will work...