Skip to content

Fix missing ENDF/B-VIII.1 TSL UinUO-*P.endf - #105

Open
yrrepy wants to merge 2 commits into
openmc-dev:masterfrom
yrrepy:ENDFB81-tsl-UinUO2-XP
Open

Fix missing ENDF/B-VIII.1 TSL UinUO-*P.endf#105
yrrepy wants to merge 2 commits into
openmc-dev:masterfrom
yrrepy:ENDFB81-tsl-UinUO2-XP

Conversation

@yrrepy

@yrrepy yrrepy commented Sep 9, 2026

Copy link
Copy Markdown

Summary

This PR adds the three uranium-in-UO₂ S(α,β) evaluations from ENDF/B-VIII.1 that are missed in processing:

  • tsl-UinUO2-5P.endf
  • tsl-UinUO2-10P.endf
  • tsl-UinUO2-100P.endf

These produce:

  • c_U_in_UO2_5p.h5
  • c_U_in_UO2_10p.h5
  • c_U_in_UO2_100p.h5

The corresponding oxygen-in-UO₂ evaluations are already processed by the existing script.

These three source evaluations use the same ZSYMAM identifier, U(UO2), as each other and as the natural-uranium evaluation. This causes the processed tables to receive conflicting names. For ENDF/B-VIII.1 processing, this PR changes only the ZSYMAM metadata field, assigning the unique identifiers UUO2-5P, UUO2-10P, and UUO2-100P before passing the evaluations to NJOY.

Validation

  • Successfully generated and reloaded all three tables with openmc.data.ThermalScattering.from_hdf5().
  • Confirmed that each table contains all eight source temperatures: 296, 400, 500, 600, 700, 800, 1000, and 1200 K.

Other: Processing Nuclide Choice

The below uranium TSL all use U238 (left as-is currently), LANL ENDF81SaB uses U235.
From my interrogation, this mostly affects the NJOY generated energy-grid that makes its way to the TSL file. Causing very small differences.
The AWR is also affected, but it doesn't seem that it is used by or from the TSL file.

  • I could see an argument for the >50% enrichment using the U235.
  • I could also see an argument for using the same nuclide as LANL (all U235).
                    ('n-092_U_238.endf', 'tsl-UinUC-100P.endf'),
                    ('n-092_U_238.endf', 'tsl-UinUC-10P.endf'),
                    ('n-092_U_238.endf', 'tsl-UinUC-5P.endf'),
                    ('n-092_U_238.endf', 'tsl-UinUC.endf'),
                    ('n-092_U_238.endf', 'tsl-UinUC-HALEU.endf'),
                    ('n-092_U_238.endf', 'tsl-UinUC-HEU.endf'),
                    ('n-092_U_238.endf', 'tsl-UinUN-100P.endf'),
                    ('n-092_U_238.endf', 'tsl-UinUN-10P.endf'),
                    ('n-092_U_238.endf', 'tsl-UinUN-5P.endf'),
                    ('n-092_U_238.endf', 'tsl-UinUN.endf'),
                    ('n-092_U_238.endf', 'tsl-UinUN-HALEU.endf'),
                    ('n-092_U_238.endf', 'tsl-UinUN-HEU.endf'),
                    ('n-092_U_238.endf', 'tsl-UinUO2-100P.endf'),
                    ('n-092_U_238.endf', 'tsl-UinUO2-10P.endf'),
                    ('n-092_U_238.endf', 'tsl-UinUO2-5P.endf'),
                    ('n-092_U_238.endf', 'tsl-UinUO2.endf'),
                    ('n-092_U_238.endf', 'tsl-UinUO2-HALEU.endf'),
                    ('n-092_U_238.endf', 'tsl-UinUO2-HEU.endf'),
                    ('n-092_U_238.endf', 'tsl-U-metal-100P.endf'),
                    ('n-092_U_238.endf', 'tsl-U-metal-10P.endf'),
                    ('n-092_U_238.endf', 'tsl-U-metal-5P.endf'),
                    ('n-092_U_238.endf', 'tsl-U-metal.endf'),
                    ('n-092_U_238.endf', 'tsl-U-metal-HALEU.endf'),
                    ('n-092_U_238.endf', 'tsl-U-metal-HEU.endf'),

Add update_zsymam function calls for thermal scattering files:
tsl-UinUO2-5P.endf
tsl-UinUO2-10P.endf
tsl-UinUO2-100P.endf

otherwise a naming collision occurs and these are not produced
(only tsl-UinUO2.endf is)
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

Successfully merging this pull request may close these issues.

1 participant