-
Notifications
You must be signed in to change notification settings - Fork 37
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
FCCAnalyses samples #228
Comments
Thanks for the thorough report, let me give a few quick answers. For 1) and 2) The 4 might be related to this, but the posted error simply looks like a potentially broken or non-present file. The failure in 3 looks like a TTree is missing a branch or a TTree is entirely missing. This would need some further debugging to understand what is really going on. The main question for me to understand is which problem needs to be solved, i.e. what do you want to achieve. If I understand the For |
The motivation was to have an understanding what is broken where.
How can this be done? Will this effectively disable those reference collections?
I believe the problem here is the addition of dNdX to the
We keep it around to be able to get this information in cases where
That would be nice to have, but is not a priority. |
Hi,
I've tested the samples we use for the studies with the several tools which might be used to understand what is saved in those files. Most important one being
podio-dump
others are good to have. The test was done on CentOS7 machine and files are stored on EOS.podio-dump
spring2021
winter2023
edm4hep2json
spring2021
winter2023
collInfo
spring2021
winter2023
Test files
spring2021
/eos/experiment/fcc/ee/generation/DelphesEvents/spring2021/IDEA/p8_ee_ZH_ecm240/events_101027117.root
winter2023
/eos/experiment/fcc/ee/generation/DelphesEvents/winter2023/IDEA/p8_ee_ZZ_ecm240/events_092194859.root
Errors
In #227 the ROOTLegacyReader is used if legacy file is detected and
following crash occures:
The text was updated successfully, but these errors were encountered: