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

Add CI #9

Open
3 tasks
clelange opened this issue Sep 26, 2023 · 3 comments
Open
3 tasks

Add CI #9

clelange opened this issue Sep 26, 2023 · 3 comments

Comments

@clelange
Copy link
Member

  • Add small ROOT file with histograms that can be used for plotting
  • Create script to use this ROOT file to create plots from
  • Use this script in CI (GitLab CI or GitHub Actions or both)
@andrzejnovak
Copy link
Contributor

andrzejnovak commented Sep 26, 2023

Better to depend on skhep_data for test instead of a committed ROOT file eghttps://github.com/scikit-hep/mplhep/blob/master/tests/test_inputs.py#L41

Can also just create TH1s on the fly with uproot ROOT :).

@clelange
Copy link
Member Author

Thanks for the pointer, but I'm not sure, it adds a dependency that might make it more difficult for people to contribute. Would surely need "contributing" docs.

@andrzejnovak
Copy link
Contributor

I'm happy to help with image comparison tests in CI also. I've set them up for mplhep, but perhaps we set these up after we settle all the style tweaks.

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

2 participants