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

STL example code #126

Merged
merged 7 commits into from
Jul 16, 2024
Merged

STL example code #126

merged 7 commits into from
Jul 16, 2024

Conversation

ejyoo921
Copy link
Contributor

I noticed that the example code for the STL test main file needed some edits, as EB2::Build has been updated. The current example inputs file is not compatible with the latest AMReX codes.

…oduce a plot of a geometry with an stl file. The inputs file also has been updated accordingly.
@ejyoo921
Copy link
Contributor Author

The generated plot file should give you the volume of airfoil.
Screenshot 2024-06-27 at 4 58 21 PM


eb2.stl_scale = 1 # default is 1
eb2.stl_center = 0 0 0 # default is (0,0,0)
eb2.stl_reverse_normal = 1 # default is 0. use "1" to see the volume of the stl object
Copy link
Member

Choose a reason for hiding this comment

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

Since this is in amrex-tutorials and this parameter stl_reverse_normal is rarely used, we might want to remove this. To "see" the volume of the stl object, we could make a new MultiFab in the code to store 1-volfrac.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I edited the main file to draw the stl object. However, I suggest keeping the eb2.stl_reverse_normal option in the input file to inform users that this parameter exists. We have used this in some PeleC test cases.

ExampleCodes/EB/STLtest/inputs Outdated Show resolved Hide resolved
ExampleCodes/EB/STLtest/inputs Outdated Show resolved Hide resolved
ExampleCodes/EB/STLtest/main.cpp Outdated Show resolved Hide resolved
@WeiqunZhang WeiqunZhang merged commit d7a0662 into AMReX-Codes:main Jul 16, 2024
14 checks passed
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