Skip to content

Foil Analysis Initial Add#24

Merged
RemDelaporteMathurin merged 12 commits intoLIBRA-project:mainfrom
cdunn314:main
Sep 19, 2025
Merged

Foil Analysis Initial Add#24
RemDelaporteMathurin merged 12 commits intoLIBRA-project:mainfrom
cdunn314:main

Conversation

@cdunn314
Copy link
Copy Markdown
Contributor

Activation foil analysis for Run 4, which will be a little different than other runs since this run had two irradiations each with their own activation foil sets and separate check source measurements.

@RemDelaporteMathurin
Copy link
Copy Markdown
Member

@cdunn314 the CI is failing because the of changes in libra-toolbox regarding openmc, i'll fix that later today and then rebase your branch

@RemDelaporteMathurin
Copy link
Copy Markdown
Member

@cdunn314 is this waiting for #LIBRA-project/libra-toolbox#86 ?

@cdunn314
Copy link
Copy Markdown
Contributor Author

@cdunn314 is this waiting for #LIBRA-project/libra-toolbox#86 ?

No, I still need to do work on the data processing as this run is a bit complicated since it's actually two runs.

BUT, the PR you mentioned and other work on using the Maximum A Posteriori (MAP) method could lead to getting some better error propagation on the activation foil analysis for all the BABY runs. See LIBRA-project/ngen_characterization#1 for more info.

@RemDelaporteMathurin
Copy link
Copy Markdown
Member

Let us know if it's easier to split it in two runs.

ping @veliki-filozof

@cdunn314
Copy link
Copy Markdown
Contributor Author

cdunn314 commented Sep 5, 2025

@RemDelaporteMathurin @veliki-filozof I've updated everything to be able to process two different irradiations each with their own activation foil sets in the same notebook, but now I'm not sure how to incorporate it into the tritium_model and processed_data.json.

The resulting neutron rates from each irradiation are pretty close together, so it wouldn't be too bad to just one average neutron rate, but since we have complete data for both irradiations, I think it would be better to use each neutron rate separately in the tritium model for each irradiation. What do you recommend?

@RemDelaporteMathurin
Copy link
Copy Markdown
Member

@cdunn314 right now, the tritium model in libra-toolbox doesn't allow for varying neutron rate. This will be possible once we upgrade the model to pathsim. So for now, I would do an average, and add a note somewhere in the tritium analysis workflow.

@RemDelaporteMathurin
Copy link
Copy Markdown
Member

@cdunn314 can you re-run the tritium analysis notebook and push it here please?

@cdunn314
Copy link
Copy Markdown
Contributor Author

@cdunn314 can you re-run the tritium analysis notebook and push it here please?

Done

@cdunn314
Copy link
Copy Markdown
Contributor Author

@cdunn314 right now, the tritium model in libra-toolbox doesn't allow for varying neutron rate. This will be possible once we upgrade the model to pathsim. So for now, I would do an average, and add a note somewhere in the tritium analysis workflow.

Now using the average of the two irradiations, and I added a note in the notebook about this

@RemDelaporteMathurin RemDelaporteMathurin merged commit 1dfaa0c into LIBRA-project:main Sep 19, 2025
1 check passed
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.

2 participants