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

Conjugate heat transfer migration as geometries #994

Draft
wants to merge 25 commits into
base: write-geoms-migration
Choose a base branch
from

Conversation

jginternational
Copy link
Member

No description provided.

@jginternational jginternational changed the base branch from master to write-geoms-migration October 10, 2024 16:23
@jginternational
Copy link
Member Author

@rubenzorrilla we need to check this one.
It writes input files but I think that calculation crashes, so probably there is something wrong there.
cht3.gid.zip

@rubenzorrilla
Copy link
Member

This one should work with the changes I'm about to push to the Kratos repo cht3_RZ.zip

@jginternational
Copy link
Member Author

solo queda revisar este modeler
image

@jginternational
Copy link
Member Author

@rubenzorrilla please check this case

aaa.gid.zip

@rubenzorrilla
Copy link
Member

@rubenzorrilla please check this case

aaa.gid.zip

As far as I see, the section

            },{
                "model_part_name" : "FluidThermalModelPart.Fluid_Left_Wall",
                "condition_name"  : "ThermalFace2D2N"

in the CreateEntitiesFromGeometriesModeler shouldn't be there. Aside of this I don't see anything strange.

However, when I ran the case I got zero residual every time step. Is the case supposed to work or it is a random one just for the sake of checking the I/O?

@jginternational
Copy link
Member Author

@rubenzorrilla it comes from the 2nd example in the GUI

@rubenzorrilla
Copy link
Member

@rubenzorrilla it comes from the 2nd example in the GUI

Then it should work, meaning that I missed something else. I'll try to have another look ASAP.

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

Successfully merging this pull request may close these issues.

3 participants