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

Tests for ThePEG #107

Closed
dberzano opened this issue Sep 22, 2015 · 5 comments
Closed

Tests for ThePEG #107

dberzano opened this issue Sep 22, 2015 · 5 comments
Assignees

Comments

@dberzano
Copy link
Contributor

We currently have the thepeg-test.sh recipe containing a simple generation done with ThePEG, based on the examples @qgp provided me with some time ago.

We would like to extend this test by adding some plots we can check to understand that everything is correct, and ultimately merge it with the aliroot-tests.sh recipe, so that they will become visible on ali-ci.cern.ch.

If you look at aliroot-tests.sh you'll notice that only .root, .sql and .log files are copied to the destination.

@qgp, could you please take care of it? We think this will be useful for everybody :-)

@qgp
Copy link
Member

qgp commented Sep 22, 2015

@dberzano: Will do, sounds good to me. And I anyway have to test the new version of ThePEG, so it's a good moment. I'll have to see in what format we can get the output (e.g. if we use Rivet for testing).

@dberzano
Copy link
Contributor Author

Any chance you use ROOT for doing the plots? ROOT is available in the aliroot-tests.sh environment.

Note: as you have noticed already, issue your PRs to IB/v5-06/prod, as master has been deleted.

@qgp
Copy link
Member

qgp commented Sep 22, 2015

Point taken, I'll try with ROOT then, i.e. I would need at least AliRoot to have the HepMC parsing, and probably AliPhysics to have some task to test. I'll have a look how much I can strip it down - while retaining the usefulness of the test, of course.

@ktf
Copy link
Member

ktf commented Jan 7, 2016

Happy New Year!
Any update on this? If not I will close it and you can reopen it when you have time to work on it.

@qgp
Copy link
Member

qgp commented Jan 7, 2016

Happy New Year!
If I am realistic, I won't get this done in January. So, let's close it now and I'll reopen it later.

@ktf ktf closed this as completed Jan 7, 2016
sawenzel pushed a commit to sawenzel/alidist that referenced this issue Apr 7, 2022
Using a MC Json for ITSTPCmatching QC
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

No branches or pull requests

3 participants