-
Notifications
You must be signed in to change notification settings - Fork 30
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
convert_fendl script broken #6
Comments
@shimwell any ideas? |
Looks like IAEA changed something in their configuration such that the download needs to appear as though it's coming from a browser. Unfortunately, that's not something that normal CI would have caught (unless we were running nightly tests or something). Anyway, I'll put in a fix (this requires something extra in OpenMC's |
Thats my point, maybe we should be running nightly tests that tries to build and get all the data, each one as a different element in the matrix? |
Interesting, I didn't think this was possible with Travis but evidently it is. The problem we'll run into is that the "generate" scripts actually run NJOY for all evaluations which takes a long time (longer than the Travis job time limits). |
This was fixed by #7. |
As in #5 CI mayve stopped this, but it seems that convert_fendl is broken - I get this error
When I run. Also could be user error, if it is let me know what Im doing wrong
The text was updated successfully, but these errors were encountered: