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

Turek & Hron FSI3 solid-calculix #300

Open
wants to merge 9 commits into
base: develop
Choose a base branch
from
Open

Conversation

JoSchrdr
Copy link

The CalculiX files from my successful FSI3 case with OpenFOAM and CalculiX.

It has to be noted, that the z-direction from the mesh is smaller than the one from the tutorial case with deal.ii.
Also, there may be some differences regarding the OpenFoam side. I can upload those too.

Copy link
Member

@MakisH MakisH left a comment

Choose a reason for hiding this comment

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

Thanks a lot for contributing! This is something we always wanted to have.

I added a few comments. Most of them should be very easy to implement, but we need to convert the case to use Stress instead of Force. I am not sure how to do that in CalculiX.

@@ -0,0 +1,30 @@
*INCLUDE, INPUT=turekflapcurved.msh
Copy link
Member

Choose a reason for hiding this comment

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

What does the "curved" part mean here?

It would be nice to document this case in the README.md of the turek-hron-fsi3/ directory.

Copy link
Author

Choose a reason for hiding this comment

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

It was just a personal tag for the different meshes that I've been used.
I will rename it.

. ../../tools/cleaning-tools.sh

clean_calculix .
rm log-solid
Copy link
Member

Choose a reason for hiding this comment

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

Where does the log-solid come from? I don't see this in the run.sh.

In any case, when calling rm, it is a good idea to call it as rm -f, so that it does not complain when the file is not there.

Copy link
Author

Choose a reason for hiding this comment

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

My bad! The log-solid came from the batch script of the cluster. I will update the run.sh file.

@@ -0,0 +1,4 @@
#!/bin/sh
Copy link
Member

Choose a reason for hiding this comment

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

The run.sh and clean.sh files are not marked as executable at the moment. Please do chmod +x run.sh clean.sh.

Copy link
Author

Choose a reason for hiding this comment

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

Alright, I don't know why it was not adopted.

Copy link
Member

Choose a reason for hiding this comment

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

Did you at any step compress these files into a .zip? I have noticed in the past that this removed file permissions.

interfaces:
- nodes-mesh: Solid-Mesh
patch: surface
read-data: [Force]
Copy link
Member

Choose a reason for hiding this comment

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

Running this case, I get:

ERROR:  Data with name "Force" is not defined on mesh "Solid-Mesh". Please add <use-data name="Force"/> under <mesh name="Solid-Mesh"/>.

Note that this case now uses stress instead of force. This will probably need a bit of work to convert, but it should be doable.

Copy link
Author

Choose a reason for hiding this comment

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

That is right, I have to dig a bit deeper for this. But first I would like to get the case running.

Copy link
Member

Choose a reason for hiding this comment

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

Don't hesitate to ask for help, this is almost there and we could merge it soon! 😄

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