Skip to content
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

Fix processes streamer #357

Merged
merged 22 commits into from
Dec 30, 2022
Merged

Fix processes streamer #357

merged 22 commits into from
Dec 30, 2022

Conversation

jgalan
Copy link
Member

@jgalan jgalan commented Dec 24, 2022

jgalan Medium: 100

see: https://root-forum.cern.ch/t/checkbytecount-and-streamer-error/52716/58

Fixes issue #348 for axionlib event data processing. However, there are still problems under certain conditions. When processing restG4 data, processes will not be present on the streamer. See new issue created to track this problem #358

  • Adds a validation pipeline in TREX to check for the number of processes written to the streamer. Still there are some problems here, not all the processes streamer are properly written to disk. See for example the processes that should be present after 03_hits.rml, TRestDetectorSignalToHitsProcess and TRestDetectorHitsAnalysisProcess are not present!

@jgalan jgalan marked this pull request as ready for review December 28, 2022 12:01
@jgalan jgalan merged commit 9bfed7c into master Dec 30, 2022
@jgalan jgalan deleted the fix_processes_streamer branch December 30, 2022 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Problem in schema evolution for processes. CheckByteCount warning/error
3 participants