Skip to content

Conversation

@n01r
Copy link
Member

@n01r n01r commented Nov 7, 2025

This PR adds a workflow to the docs that describes how to work with multiple STL files and unify them for import into WarpX where they provide the configuration for embedded boundaries.

Credit goes to @agargone who authored the docs.

n01r and others added 3 commits November 7, 2025 13:59
Create a dummy documentation page where we will explain the workflow to
make a union out of several STL files and then make the file watertight
to make it a valid import for WarpX.
Fixed Part 2

Fixed indentation

slight typo fix

Fixed links

Fixed links

Fixed links

Fixed links

Removed further reading and added small comment about resolution

Removed suggested tools
@n01r n01r added component: documentation Docs, readme and manual hackathon Let's address this topic during the GPU hackathon component: boundary PML, embedded boundaries, et al. labels Nov 7, 2025
@n01r n01r requested review from RemiLehe and SahelHakimi November 7, 2025 22:55
2. The STL geometry is "watertight" (The STL file shouldn't have duplicate faces or vertices, overlapping vertices should be merged, and there shouldn't be any gaps or holes in the mesh)
3. The STL file is properly formatted for WarpX input

.. 2. The STL file shouldn't have duplicate faces or vertices, and overlapping vertices should be merged. (issues with vertices can cause the MLMG solver to crash)
Copy link
Member

Choose a reason for hiding this comment

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

@n01r cleanup commented out text? :)

Comment on lines +83 to +84
.. * ``Filters -> Cleaning and Repairing -> Remove Zero Area Faces``
.. * ``Filters -> Remeshing, Simplification and Reconstruction -> Close Holes``
Copy link
Member

Choose a reason for hiding this comment

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

@n01r cleanup commented out text? :)

Copy link
Member

@ax3l ax3l left a comment

Choose a reason for hiding this comment

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

LGTM, thank you! :)

@ax3l ax3l enabled auto-merge (squash) November 17, 2025 17:47
@ax3l ax3l merged commit c4e7315 into BLAST-WarpX:development Nov 17, 2025
45 of 46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: boundary PML, embedded boundaries, et al. component: documentation Docs, readme and manual hackathon Let's address this topic during the GPU hackathon

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants