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

Semi-coherent zdc benchmark #18

Merged
merged 14 commits into from
Nov 19, 2024
Merged

Semi-coherent zdc benchmark #18

merged 14 commits into from
Nov 19, 2024

Conversation

JiaJunHuang120000
Copy link
Contributor

Briefly, what does this PR introduce?

Semi-coherent events analysis for the zdc lyso.

What kind of change does this PR introduce?

  • Bug fix (issue #__)
  • New feature (issue #__)
  • Documentation update
  • Other: Added one plot for the energy spectrum of photons in the zdc lyso

Please check if this PR fulfills the following:

  • Tests for the changes have been added
  • Documentation has been added / updated
  • Changes have been communicated to collaborators @bschmookler

Does this PR introduce breaking changes? What changes might users need to make to their code?

No.

Does this PR change default behavior?

No.

@veprbl
Copy link
Member

veprbl commented Oct 3, 2024

@JiaJunHuang120000 have you tested this? I'm getting an error in abconv, also it's on CI:
https://eicweb.phy.anl.gov/EIC/benchmarks/physics_benchmarks/-/jobs/3913962

@bschmookler
Copy link
Contributor

@JiaJunHuang120000 have you tested this? I'm getting an error in abconv, also it's on CI: https://eicweb.phy.anl.gov/EIC/benchmarks/physics_benchmarks/-/jobs/3913962

The problem seems to be due to the BeAGLE file on S3/xrdroot. It is an older BeAGLE file. If we generate a BeAGLE file ourselves locally, convert it to HepMC format, and then run abconv, everything works fine.

The only eA BeAGLE file we can find on S3/xrdroot is this one:

xrdfs root://dtn-eic.jlab.org ls /work/eic2/EPIC/EVGEN/CI/ePb_beagle.hepmc
/work/eic2/EPIC/EVGEN/CI/ePb_beagle.hepmc

Are there any other ones available? Or can we have write permission to upload some BeAGLE files that we generated?

@veprbl
Copy link
Member

veprbl commented Oct 8, 2024

@bschmookler Wild idea: I assume we have beagle installed in the container. Could you script hepmc3 generation into the benchmark?

@bschmookler
Copy link
Contributor

I don't see it in /opt/local/bin/. Would it be somewhere else in the container?

@veprbl
Copy link
Member

veprbl commented Oct 8, 2024

Okay, I now remember. We can't get beagle in the container if it depends on fluka. Can you share your file then? I can get it onto XRootD myself.

@JiaJunHuang120000
Copy link
Contributor Author

Hi,

I have updated the snakefile to take files from JLab XRootD, but there is some changes to the \master branch which I am not sure how to account for. Please let me know what is different. Thanks!

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using textual representation of HepMC3 is not a good idea. Please, re-write this using

import HepMC3

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the module install inside ./eic-shell? I receive an error while importing ModuleNotFoundError: No module named 'HepMC3'.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, looks like it's import pyHepMC3

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi,

Can we merge this branch first now that and changes have been settle, and later on we can modify the for using the module?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, go ahead.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@JiaJunHuang120000 Any follow up on this?

@JiaJunHuang120000 JiaJunHuang120000 merged commit ac67c33 into master Nov 19, 2024
2 checks passed
@JiaJunHuang120000 JiaJunHuang120000 deleted the zdc_lyso_physics branch November 19, 2024 17:05
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.

3 participants