coresoftware icon indicating copy to clipboard operation
coresoftware copied to clipboard

DRAFT PR DO NOT MERGE: Adding TpcTpotEventInfo to the pool manager

Open rosstom2232 opened this issue 2 years ago • 3 comments

Types of changes

  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to not work for users)
  • [ ] Requiring change in macros repository (Please provide links to the macros pull request in the last section)
  • [x] I am a member of GitHub organization of sPHENIX Collaboration, EIC, or ECCE (contact Chris Pinkenburg to join)

What kind of change does this PR introduce? (Bug fix, feature, ...)

TODOs (if applicable)

Links to other PRs in macros and calibration repositories (if applicable)

rosstom2232 avatar Oct 21 '23 01:10 rosstom2232

Build & test report

Report for commit f85ca76a3581a92feb6556228ec3216a987ad369: Jenkins on fire


Automatically generated by sPHENIX Jenkins continuous integration sPHENIX             jenkins.io

sphenix-jenkins-ci[bot] avatar Oct 21 '23 01:10 sphenix-jenkins-ci[bot]

Build & test report

Report for commit 9e8c44f442adc93e721dbec5c9c83db83bc0a3b2: Jenkins on fire


Automatically generated by sPHENIX Jenkins continuous integration sPHENIX             jenkins.io

sphenix-jenkins-ci[bot] avatar Oct 23 '23 22:10 sphenix-jenkins-ci[bot]

I had a look at the failure - we cannot have any dependencies on other libraries here. That can be overcome by moving those classes to ffarawobjects. But looking at this - with the pooling where the time ordering of the data is preserved this should really not be necessary anymore

pinkenburg avatar Oct 24 '23 00:10 pinkenburg