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

zero awp values for non photon particles in MF6 cause HEATR crash #144

Closed
whaeck opened this issue Jan 16, 2020 · 7 comments
Closed

zero awp values for non photon particles in MF6 cause HEATR crash #144

whaeck opened this issue Jan 16, 2020 · 7 comments
Assignees

Comments

@whaeck
Copy link
Member

whaeck commented Jan 16, 2020

It was brought to our attention that a division by zero occurred in HEATR with the Be10 evaluation from TENDL2017:

 njoy 2016.53  07Nov19                                       01/16/20 14:21:40
 *****************************************************************************

 moder...                                                                 0.0s

 reconr...                                                                0.1s

 ---message from rdf2bw---calculation of angular distribution not installed.

 broadr...                                                                0.1s
      293.0 deg                                                           0.1s
      600.0 deg                                                           0.2s
      900.0 deg                                                           0.2s

 heatr...                                                                 0.2s

 ---message from hinit---mf6, mt102 does not give recoil za=  4011
                          photon momentum recoil used.

Program received signal SIGFPE: Floating-point exception - erroneous arithmetic operation.

Backtrace for this error:
#0  0x7f2d8c7123ef in ???
#1  0x7f2d8d946dfc in df
	at /home/wim/Projects/NJOY2016/njoy2016-photoatomic/src/heatr.f90:2031
#2  0x7f2d8d93fa56 in sixbar
	at /home/wim/Projects/NJOY2016/njoy2016-photoatomic/src/heatr.f90:2835
#3  0x7f2d8d94ab53 in nheat
	at /home/wim/Projects/NJOY2016/njoy2016-photoatomic/src/heatr.f90:1653
#4  0x7f2d8d958164 in __heatm_MOD_heatr
	at /home/wim/Projects/NJOY2016/njoy2016-photoatomic/src/heatr.f90:375
#5  0x401ca8 in njoy
	at /home/wim/Projects/NJOY2016/njoy2016-photoatomic/src/main.f90:205
#6  0x401f01 in main
	at /home/wim/Projects/NJOY2016/njoy2016-photoatomic/src/main.f90:112
Floating point exception

This can be reproduced with the following ENDF file and input file:
input2.txt
n-Be10-tendl2017.endf.txt

@whaeck
Copy link
Member Author

whaeck commented Jan 16, 2020

The issue occurs in the df function (line 2004 in the heatr.f90 file) which is used to calculate damage energy. The division by zero occurs in the following lines (line 2030-2031):

      denom=(zr**twothd+zl**twothd)**threeq*ar**onep5*sqrt(al)
      fl=c2*zr**twothd*sqrt(zl)*(ar+al)**onep5/denom

The value of ar appears to be zero for a non photon secondary particle, these values are read directly from the evaluation.

The evaluation in question appears to have zero atomic weight ratio values for some of the MF6 subsections. This appears to happen for the following MTs and particle subsections in MF6:
MT22, particle 2006
MT24, particle 2005
MT106, particle 2008
MT107, particle 2007

@whaeck
Copy link
Member Author

whaeck commented Jan 16, 2020

A branch was created (https://github.com/njoy/NJOY2016/tree/fix/tendl-zero-awr) to detect this issue in the future and to make NJOY exit gracefully. As this is an issue related to an error in an evaluation, we cannot fix this problem.

I'll do a proper pull request next week.

@jchsublet You may want to take note of this one.

@jchsublet
Copy link

@whaeck I will thank you but you may also take this to the true evaluator(s) for B010 and B011, much nearer to you, as they are copy of ENDF/B-VIII.O in TENDL-2017, and TENDL-2019

@paulromano
Copy link
Member

The original issue said Be10. Are we talking about Be10 or B10?

@kahlerac
Copy link
Contributor

kahlerac commented Jan 17, 2020 via email

@jchsublet
Copy link

@paulromano Mea culpa Be isotopes it was an no B !! and for Be10 and Be11 TENDL-2017 MF6 is incorrect as Skip pointed out, but then my very similar HEATR sequence did not crashed, having said that division by 0 and FFLAGS have a said in the matter. So the for the above the files, the evaluation are at faults, definitely

As TENDL-2017 has just been superseded by TENDL-2019 (That's include quite a few format correction) I have checked the above input2.txt sequence with NJOY2016.53 and its works without crashing for TENDL-2019, Be010, Be011 and CO14. More complex HEATR sequences were achieved in the production of the Ace-TENDL-2019 library.

@whaeck
Copy link
Member Author

whaeck commented Jan 22, 2020

This will now be detected in NJOY2016.

@whaeck whaeck closed this as completed Jan 22, 2020
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

4 participants