-
Notifications
You must be signed in to change notification settings - Fork 87
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
Fix/mf35 sequence #287
Fix/mf35 sequence #287
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a documentation nitpick. This works on the affected files for me, and tests seem to pass.
ReleaseNotes.md
Outdated
@@ -4,6 +4,7 @@ Given here are some release notes for NJOY2016. Each release is made through a f | |||
## [NJOY2016.70](https://github.com/njoy/NJOY2016/pull/xxx) | |||
This update fixes a number of minor issues: | |||
- Fixed an issue in HEATR when reading evaluations with large multiplicity tables in MF6. | |||
- Multiple ERRORR calls can now be made in the same input file without crashing. This is of interest to users that wish to process MF34 and MF35 (where ERRORR needs to be called for each sub-subsection and incident energy group). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should make reference to this affecting files with LRF7 resonances without MF32 data.
@nathangibson14 need you to reapprove |
@nathangibson14 You may want to reapprove this one |
Properly deallocating sammy arrays solves the issue when running multiple ERRORR isntances in the same input.