Skip to content

New way of evaluating drum material properties for thermal conduction#790

Open
YaqiWang wants to merge 1 commit into
idaholab:develfrom
YaqiWang:drum_thermal_prop
Open

New way of evaluating drum material properties for thermal conduction#790
YaqiWang wants to merge 1 commit into
idaholab:develfrom
YaqiWang:drum_thermal_prop

Conversation

@YaqiWang
Copy link
Copy Markdown
Contributor

@YaqiWang YaqiWang commented Apr 15, 2026

The new material proposed in MOOSE (idaholab/moose#32773) is still under review. I also need to run the model to make sure the correctness of the change. This closes #797.

@YaqiWang
Copy link
Copy Markdown
Contributor Author

Note that we do not save much number of input lines wise here because there is only one drum in this model and only two segments on the drum. It does make users avoid the mind boggling math in the functions. But in a full core case, we will have 12 drums and this change can help.

@YaqiWang YaqiWang force-pushed the drum_thermal_prop branch from 8698da7 to 0d88a73 Compare April 15, 2026 23:52
@YaqiWang YaqiWang force-pushed the drum_thermal_prop branch 2 times, most recently from e1a270c to 2b27c0b Compare April 23, 2026 19:52
@YaqiWang
Copy link
Copy Markdown
Contributor Author

@GiudGiud just to let you know that this is updated according to the merged change in MOOSE.

Comment thread microreactors/drum_rotation/thermal_transient.i Outdated
@YaqiWang YaqiWang force-pushed the drum_thermal_prop branch 4 times, most recently from 3ed265a to d5c2ddb Compare April 24, 2026 16:30
@YaqiWang YaqiWang force-pushed the drum_thermal_prop branch from d5c2ddb to 09b237b Compare May 12, 2026 17:38
@YaqiWang YaqiWang changed the title WIP: New way of evaluating drum material properties for thermal conduction New way of evaluating drum material properties for thermal conduction May 12, 2026
@moosebuild
Copy link
Copy Markdown

moosebuild commented May 12, 2026

Job VTB Documentation, step Sync Docs on 5d0ee43 wanted to post the following:

View the site here

This comment will be updated on new commits.

@GiudGiud
Copy link
Copy Markdown
Collaborator

doco error:

MooseDocs.extensions.tagging (ForkProcess-7): index.md: Provided 'key' not in allowed_keys (see config.yml); not adding the following to the database: V&V
MooseDocs.base.readers (ForkProcess-9): 
TOKENIZE ERROR: Unable to find block 'Functions' in /tmp/build/virtual_test_bed/microreactors/drum_rotation/thermal_ss.i.
/tmp/build/virtual_test_bed/doc/content/microreactors/drum_rotation/index.md:208
   ┌───────────────────────────────────────────────────────────────────────────────────────────────┐
208│                                                                                               │
209│                                                                                               │
210│!listing microreactors/drum_rotation/thermal_ss.i                                              │
211│  block=Functions                                                                              │
212│  caption=Functions for thermal properties in drum region                                      │
213│  id=lst:thermal_drum    

@YaqiWang YaqiWang force-pushed the drum_thermal_prop branch from 09b237b to 5d0ee43 Compare May 12, 2026 19:16
@YaqiWang
Copy link
Copy Markdown
Contributor Author

doco error:

MooseDocs.extensions.tagging (ForkProcess-7): index.md: Provided 'key' not in allowed_keys (see config.yml); not adding the following to the database: V&V
MooseDocs.base.readers (ForkProcess-9): 
TOKENIZE ERROR: Unable to find block 'Functions' in /tmp/build/virtual_test_bed/microreactors/drum_rotation/thermal_ss.i.
/tmp/build/virtual_test_bed/doc/content/microreactors/drum_rotation/index.md:208
   ┌───────────────────────────────────────────────────────────────────────────────────────────────┐
208│                                                                                               │
209│                                                                                               │
210│!listing microreactors/drum_rotation/thermal_ss.i                                              │
211│  block=Functions                                                                              │
212│  caption=Functions for thermal properties in drum region                                      │
213│  id=lst:thermal_drum    

Yeah, I should have deleted this because functions are no longer used.

@GiudGiud
Copy link
Copy Markdown
Collaborator

We'll wait on a Griffin submodule with an updated MOOSE to merge this, don't want to add a temporary test failure right now

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.

Improve microreactor drum rotation model with the new ControlDrumMaterial

3 participants