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! :)

tests/test_eb.py Outdated
Comment on lines 10 to 14
pass

# TODO:
# EB2_Build(...)
# makeEBFabFactory(...)
Copy link
Member

@ax3l ax3l Nov 7, 2025

Choose a reason for hiding this comment

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

@WeiqunZhang do you like to finalize your PR by adding a test that uses your new functionality, so we have test coverage?

Suggested change
pass
# TODO:
# EB2_Build(...)
# makeEBFabFactory(...)
... = EB2_Build(...)
... = makeEBFabFactory(...)

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, I will do it.

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