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

Streaming Hepmc3 ROOT input from XRootD is broken in 01-29 #1281

Closed
1 task done
veprbl opened this issue Jun 24, 2024 · 5 comments · Fixed by #1283
Closed
1 task done

Streaming Hepmc3 ROOT input from XRootD is broken in 01-29 #1281

veprbl opened this issue Jun 24, 2024 · 5 comments · Fixed by #1283
Assignees
Labels

Comments

@veprbl
Copy link
Contributor

veprbl commented Jun 24, 2024

Check duplicate issues.

  • Checked for duplicates

Goal

Stream big files without downloading them locally

Operating System and Version

doesn't matter

compiler

doesn't matter

ROOT Version

doesn't matter

DD4hep Version

01-29

Reproducer

ddsim   --runType batch  --filter.tracker edep0   -v WARNING   --numberOfEvents 100   --compactFile $DETECTOR_PATH/epic.xml   --inputFiles root://dtn-eic.jlab.org//work/eic2/EPIC/EVGEN/DIS/NC/10x100/minQ2=1/pythia8NCDIS_10x100_minQ2=1_beamEffects_xAngle=-0.025_hiDiv_1.hepmc3.tree.root   --outputFile sim_output/epic/backgrounds/EPIC/EVGEN/DIS/NC/10x100/minQ2=1/pythia8NCDIS_10x100_minQ2=1_beamEffects_xAngle=-0.025_hiDiv_1.edm4hep.root

gives

ERROR: The inputfile 'root://dtn-eic.jlab.org//work/eic2/EPIC/EVGEN/DIS/NC/10x100/minQ2=1/pythia8NCDIS_10x100_minQ2=1_beamEffects_xAngle=-0.025_hiDiv_1.hepmc3.tree.root' does not exist

Additional context

The likely cause for regression is in da34149 (#1247)

@veprbl veprbl added the bug label Jun 24, 2024
@andresailer
Copy link
Member

Just not check when the URL starts with anything but / or is there some xrd stat call we could make instead?

@wdconinc
Copy link
Contributor

xrdfs stat, e.g.

xrdfs dtn-eic.jlab.org stat /work/eic2/EPIC/EVGEN/BACKGROUNDS/BEAMGAS/proton/pythia8.306-1.0/100GeV/pythia8.306-1.0_ProtonBeamGas_100GeV_run001.hepmc3.tree.root

@andresailer
Copy link
Member

@veprbl
Copy link
Contributor Author

veprbl commented Jun 25, 2024

Can we forego check for XRootD or even all URLs completely? It makes no sense to impose a separate mechanism that doesn't have the same behaviors as TFile, that will lead to failures in some other edge cases.

@andresailer
Copy link
Member

Please see #1283.

@andresailer andresailer self-assigned this Jun 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants