McParticle: moving default to 001#8048
McParticle: moving default to 001#8048jgrosseo merged 1 commit intoAliceO2Group:devAliceO2Group/AliceO2:devfrom
Conversation
|
@nburmaso I have removed the duplication of the table definition. Do I overlook something or is it not needed actually? |
|
This PR depends on AliceO2Group/O2Physics#488 |
Hi @jgrosseo. I implemented a separate table for MC particles so that I had a table cursor without dynamic columns in it. At least it was needed at the moment when I first added MC particles to the converter. I guess |
|
AliceO2Group/O2Physics#488 breaks the analysis testing which is done inside the full system test. This is unfortunately still the case with the PR here. I see in the fullCI log: where the first seems to be a simple transition to the new table on the O2Physics side. The second one is a segfault. |
Do you know what needs to be fixed? |
sawenzel
left a comment
There was a problem hiding this comment.
Ok let's merge it. FullCI is back to normal in a local test.
|
Remaining issues followed up here: https://alice.its.cern.ch/jira/browse/O2-2766 |
|
Thanks a lot! |
|
I have asked @ktf for a daily tag. Then we can merge AliceO2Group/O2Physics#491 and all tests should be back in business. |
No description provided.