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

NJOY2016.57 88-Ra-226g.jeff33 parsing error #165

Open
njchoi opened this issue Jun 4, 2020 · 10 comments
Open

NJOY2016.57 88-Ra-226g.jeff33 parsing error #165

njchoi opened this issue Jun 4, 2020 · 10 comments

Comments

@njchoi
Copy link

njchoi commented Jun 4, 2020

Hello.
Thank you for the replies in the ENDF/B-VIII.0 B10 issue before.
This time I brought an issue regarding JEFF-3.3.
Followings are the input I used and a segment of the output ACE file.
The processing finishes normally, but a strange value is observed (2.12448227712-322) and my ACE parser breaks down.
It is not even in the proper scientific notation.
This is located in the NU block of the 6th delayed neutron precursor group.
Could you please take a look at it?
Many thanks in advance!

reconr
20 21/
'pendf tape for 88-Ra-226g.jeff33'/
8834 0/
.001/
0/
broadr
20 21 22/
8834 1/
.001/
1800/
0/
acer
20 22 0 23 0/
1/
'reprocessed from 88-Ra-226g.jeff33'/
8834 1800/
/
/
stop

image

@paulromano
Copy link
Member

NJOY does occasionally produce values that have a three-digit exponent. When this happens, the ACE file written will omit the 'e' in the exponential notation. This does cause pain if you're writing an ACE parser -- I know from first-hand experience. You can take a look at I did in OpenMC to handle this case here:
https://github.com/openmc-dev/openmc/blob/develop/openmc/data/ace.py#L408-L417

^That may help if you're writing Python. For a solution in C, see here:
https://github.com/openmc-dev/openmc/blob/develop/openmc/data/endf.c

Note that the ENDF format itself does allow numbers in exponential notation to be written without the 'e' 🤦

@njchoi
Copy link
Author

njchoi commented Jun 4, 2020

Thank you, Dr. Romano. That's a new information for me and the references will be helpful.
But even considering the exponent format problem, the value does not seem to be physical.

@paulromano
Copy link
Member

Yes, to me it looks like an uninitialized value (which often tend to manifest as a value very close to zero as you have there). The NJOY devs will have to weigh in on what the real culprit here might be.

@jchsublet
Copy link

@njchoi @paulromano numerical issues in Ace are always interesting thanks for bringing this one up, but if one first take a look at the source in that case Ra226 1455, all fits into place, the wrong places in that case

0.000000+0 0.000000+0 0 0 1 08834 1455 2
1.000000+1 8834 1455 3

One group !! when it should contains 6 to be consistent with 5455. Try with

0.000000+0 0.000000+0 0 0 6 08834 1455 2
1.332900-2 3.051000-2 1.151600-1 2.974000-1 8.476600-1 2.879600+09440 1455 3

This error will happen in more targets (for ENDF/B-VIII.0)

outAm240:MF1 MT455 claims 1 delayed neutron groups, but MF5 MT455 claims 6 groups
outBk245:MF1 MT455 claims 1 delayed neutron groups, but MF5 MT455 claims 6 groups
outBk246:MF1 MT455 claims 1 delayed neutron groups, but MF5 MT455 claims 6 groups
outBk247:MF1 MT455 claims 1 delayed neutron groups, but MF5 MT455 claims 6 groups
outBk248:MF1 MT455 claims 1 delayed neutron groups, but MF5 MT455 claims 6 groups
outCf246:MF1 MT455 claims 1 delayed neutron groups, but MF5 MT455 claims 6 groups
outCf248:MF1 MT455 claims 1 delayed neutron groups, but MF5 MT455 claims 6 groups
outCf253:MF1 MT455 claims 1 delayed neutron groups, but MF5 MT455 claims 6 groups
outCm240:MF1 MT455 claims 1 delayed neutron groups, but MF5 MT455 claims 6 groups
outCm244:MF1 MT455 claims 1 delayed neutron groups, but MF5 MT455 claims 6 groups
outCm247:MF1 MT455 claims 1 delayed neutron groups, but MF5 MT455 claims 6 groups
outNp234:MF1 MT455 claims 1 delayed neutron groups, but MF5 MT455 claims 6 groups
outNp235:MF1 MT455 claims 1 delayed neutron groups, but MF5 MT455 claims 6 groups
outPa229:MF1 MT455 claims 1 delayed neutron groups, but MF5 MT455 claims 6 groups
outPa230:MF1 MT455 claims 1 delayed neutron groups, but MF5 MT455 claims 6 groups
outPu237:MF1 MT455 claims 1 delayed neutron groups, but MF5 MT455 claims 6 groups
outPu246:MF1 MT455 claims 1 delayed neutron groups, but MF5 MT455 claims 6 groups
outRa223:MF1 MT455 claims 1 delayed neutron groups, but MF5 MT455 claims 6 groups
outRa224:MF1 MT455 claims 1 delayed neutron groups, but MF5 MT455 claims 6 groups
outRa225:MF1 MT455 claims 1 delayed neutron groups, but MF5 MT455 claims 6 groups
outRa226:MF1 MT455 claims 1 delayed neutron groups, but MF5 MT455 claims 6 groups
outTh231:MF1 MT455 claims 1 delayed neutron groups, but MF5 MT455 claims 6 groups

Acer, always has been too merciful to evaluator, library assembler, thank to the commendable spirit of his father, this may change with the custodian

@njchoi
Copy link
Author

njchoi commented Jun 4, 2020

So is this a data bug rather than a software bug?

@jchsublet
Copy link

Most definitely, the nuclear data is inconsistent, the ENDF-6 format frame not obeyed. Since this is trivial to correct or get it right, why should the processing software developer provide for even a warning?

@kahlerac
Copy link
Contributor

kahlerac commented Jun 4, 2020 via email

@jchsublet
Copy link

@kahlerac not sure I get the above remark or did I missed some-thing? I do not see any issue with the "e" but more with 1 group declared in 1455, when 6 spectra are in 5455, for my palate, may be too French, 6 groups with timing should be declared in 1455, the two lines proposal

@kahlerac
Copy link
Contributor

kahlerac commented Jun 4, 2020 via email

@paulromano
Copy link
Member

Yes, it did originate in Fortran. My point was that the ENDF-6 formats manual explicitly allows e-less floats as well as floats with spaces interspersed, as you alluded to, so I would be careful to not say that it's only a Fortran language feature. If you are writing an ENDF parser in a language other than Fortran, you do need to account for this possibility to truly conform to what the ENDF manual says.

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