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

Replace OpenMCTallyNuclides test #1032

Open
pshriwise opened this issue Jan 17, 2025 · 0 comments
Open

Replace OpenMCTallyNuclides test #1032

pshriwise opened this issue Jan 17, 2025 · 0 comments

Comments

@pshriwise
Copy link
Collaborator

In #923 we've decided that tallies generated by Cardinal for mapping should not be modified by such user objects, including the OpenMCTallyEditor UOs that will replace the OpenMCTallyNuclides object (PR discussion).

Because the original test relied on a CSV output of the mapped tally data (see linked files below), we can't reproduce this test as-is without some unnecessary parsing of the tallies.out file or something. In light of that we're going to defer translation of this test until we determine how we'll provide the OpenMC Python API as part of the Cardinal installation (PR discission.

Links to original test files:
Input File
Test File

For developments on this follow #592.

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

1 participant