Skip to content

Permeation experiment with FLiBe - #270

Closed
leekadz wants to merge 13 commits into
idaholab:develfrom
leekadz:val-flibe
Closed

Permeation experiment with FLiBe#270
leekadz wants to merge 13 commits into
idaholab:develfrom
leekadz:val-flibe

Conversation

@leekadz

@leekadz leekadz commented Jun 17, 2025

Copy link
Copy Markdown
Collaborator

Reproduce Calderoni et al. (2008) tritium in FLiBe experiment
(Ref. #12)
(Ref. #193)

@simopier

@simopier simopier self-assigned this Jun 19, 2025
@simopier
simopier self-requested a review June 19, 2025 02:23
@simopier simopier added the V&V Relevant to V&V label Jun 19, 2025
@moosebuild

moosebuild commented Jun 19, 2025

Copy link
Copy Markdown

Job Documentation, step Sync to remote on 1b21cb4 wanted to post the following:

View the site here

This comment will be updated on new commits.

@simopier simopier left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!
Good first take using a very simplified version of the model to reproduce this experimental data. Obviously, there is more to do here to match the measurements, but we'll get there step by step.
I have some comments/suggestions, especially related to how the Ni/FLiBe interface is being modeled, which we should discuss.

Additionally, it seems like you are having the same issue as detailed in: #200
Look at how we temporarily updated the test specification files to not run them for these validation cases, and reproduce this here.

Comment thread doc/content/bib/tmap8.bib Outdated
Comment thread doc/content/bib/tmap8.bib Outdated
Comment thread doc/content/verification_and_validation/index.md Outdated
Comment thread doc/content/verification_and_validation/val-flibe.md Outdated
Comment thread doc/content/verification_and_validation/val-flibe.md Outdated
| --------- | ----------- | ----- | ----- | --------- |
| $l_{\mathrm{Ni}}$ | Ni membrane thickness | $2$ | mm | [!cite](calderoni2008measurement) |
| $l_{\mathrm{FLiBe}}$ | FLiBe membrane thickness | $81$ | mm | [!cite](calderoni2008measurement) |
| $P_{T_2}$ | Input pressure | $1210$ | Pa | [!cite](hattab2024openfoam) |

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is that value not from Pattrick's paper?

| $K_{s, \mathrm{FLiBe}}$ | Henry's law solubility for tritium in FLiBe | $7.9 \times 10^{-2} \exp(- 35 \times 10^3 / RT)$ | mol/m$^3$/Pa | [!cite](calderoni2008measurement) |
| $D_{\mathrm{Ni}}$ | Diffusivity of tritium in nickel | $7 \times 10^{-7} \exp(- 39.5 \times 10^3 / RT)$ | m$^2$/s | [!cite](causey2012tritium) |
| $K_{s, \mathrm{Ni}}$ | Sieverts' law solubility for tritium in nickel | $564 \times 10^{-3} \exp(- 15.8 \times 10^3 / RT)$ | mol/m$^3$/Pa$^{0.5}$ | [!cite](calderoni2008measurement) |
| $K_{d, \mathrm{Ni}}$ | Surface rate of tritium in nickel | $1.44 \times 10^{-6} \exp(- 29.68 \times 10^3 / RT)$ | mol/m$^{2}$/s/Pa | [!cite](altunoglu1991permeation) |

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are currently not using this term, are you?

Comment thread doc/content/verification_and_validation/val-flibe.md Outdated
Comment thread doc/content/verification_and_validation/val-flibe.md Outdated
Comment thread test/tests/val-flibe/val-flibe.i Outdated
Comment on lines +130 to +142
[interface_sorption_Ni_FLiBe]
type = InterfaceSorption
K0 = ${K_s_FLiBe_prefactor}
Ea = ${K_s_FLiBe_energy}
n_sorption = ${n_Henry}
diffusivity = ${D_FLiBe}
unit_scale = ${unit_scale}
unit_scale_neighbor = ${unit_scale_neighbor}
temperature = ${temperature}
variable = 'tritium_concentration_Ni'
neighbor_var = 'tritium_concentration_FLiBe'
sorption_penalty = 1e1
boundary = 'interface_Ni_FLiBe'

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This does not correspond to the documentation (the documentation is wrong).
Also, let's discuss this in general, as I don't think this is correct either.

@moosebuild

Copy link
Copy Markdown

Job Coverage, step Generate coverage on e2c9d34 wanted to post the following:

Coverage

Coverage did not change

Full coverage report

This comment will be updated on new commits.

@simopier

Copy link
Copy Markdown
Collaborator

restarted in #384.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

V&V Relevant to V&V

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants