-
Notifications
You must be signed in to change notification settings - Fork 1
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 pytest GitHub action #104
Conversation
dachengx
commented
Jan 17, 2024
•
edited
Loading
edited
- Add a file to trigger the pytest
- Add dependencies of straxen and scikit-learn
The errors are:
which I have reproduced also in the latest montecarlo environment. The errors are from |
To me this looks like the nestpy version issue. |
Yes, I wonder if the nestpy version in the base environment prevents the hardcoded install in the montecarlo_environment. |
I think it is not only a problem of base_environment. Maybe I can let GitHub action run XENONnT/montecarlo_environment#8 before testing. |
The timeout can only be solved by saving dependencies after #112. As discussed with @HenningSE and @ramirezdiego , I will merge this PR to give more time for other PRs. |
Sorry, I am wrong about the reason for failed pytests. They are simply because of too short timeout limit. It is fixed by e8f2255. |