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

Move convective heat transfer coefficient and convection/radiation reference temperatures to the 'boundary' input block #297

Open
stvdwtt opened this issue Jun 24, 2024 · 1 comment
Assignees

Comments

@stvdwtt
Copy link
Collaborator

stvdwtt commented Jun 24, 2024

These three parameters aren't fundamental material properties and are instead depend on the environment being simulated. I think these are a better fit for the boundary input block. Normally I wouldn't say this is worth changing, but it messes with Mist integration to have non-material properties in the materials block.

Any concerns about the switch @Rombur?

h/t to @hespuck19 for catching this

@stvdwtt stvdwtt self-assigned this Jun 24, 2024
@Rombur
Copy link
Member

Rombur commented Jun 24, 2024

Sure, you can move these parameters.

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

2 participants