Skip to content

Conversation

@WeiqunZhang
Copy link
Member

This is still work in progress. You can build EB and create EBFArrayBoxFactory objects from python.

This is still work in progress. You can build EB and create
EBFArrayBoxFactory objects from python.
@WeiqunZhang WeiqunZhang requested a review from ax3l November 6, 2025 21:03
@ax3l ax3l added the component: EB embedded boundaries label Nov 7, 2025
ax3l added 2 commits November 7, 2025 14:36
Just simplifies the call site.
Signed-off-by: Axel Huebl <[email protected]>
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.

Thank you for starting this! :)

ax3l added 2 commits November 7, 2025 21:36
`foreach` is not a function, so once 1D was built
all EB `.cpp` files were skipped.
Guards not needed.
@ax3l ax3l self-assigned this Nov 8, 2025
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.

Good start, thank you! ✨

@ax3l ax3l merged commit 51f8cb3 into AMReX-Codes:development Nov 8, 2025
18 checks passed
Comment on lines +289 to +295
Embedded boundary (EB) support in pyAMReX is still minimal. To build pyAMReX with
EB support, you need to add ``-DAMReX_EB=ON`` to CMake build options.

.. autofunction:: amrex.space3d.EB2_Build

.. autoclass:: amrex.space3d.EBFArrayBoxFactory
:members:
Copy link
Member

Choose a reason for hiding this comment

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

Hm, gotta build locally but something broke the autodocs for all types
https://pyamrex.readthedocs.io/en/latest/usage/api.html

Copy link
Member

Choose a reason for hiding this comment

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

I built locally and the issue is sizmailov/pybind11-stubgen#275

EBFArrayBoxFactory depends on FabFactory_FArrayBox but pybind11-stubgen does falsly an alphabetical sorting. Hoping this PR gets accepted and unbreaks it, otherwise we need to fork pybind11-stubgen.

Copy link
Member

Choose a reason for hiding this comment

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

@ax3l ax3l mentioned this pull request Nov 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: EB embedded boundaries

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants