-
Notifications
You must be signed in to change notification settings - Fork 50
Add mesh_layout argument with rectilinear two-step coordinate/connectivity architecture #81
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
Merged
leifdenby
merged 24 commits into
mllam:main
from
prajwal-tech07:feat/mesh-layout-rectilinear
May 29, 2026
Merged
Changes from all commits
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
6f82d43
feat: introduce mesh_layout argument with two-step coordinate/connect…
prajwal-tech07 63af996
test: add comprehensive tests for mesh_layout two-step architecture
prajwal-tech07 f417962
refactor: align API with Leif's final table — rename params, simplify…
prajwal-tech07 30ddb19
docs: add CHANGELOG entry for mesh_layout feature
prajwal-tech07 802b66a
Address all review feedback from PR #81
prajwal-tech07 21d33e8
refactor: add coords module with two-step mesh creation (primitives +…
prajwal-tech07 8d18530
refactor: use explicit keyword argument names in flat mesh functions
prajwal-tech07 4f5a8de
refactor: put default values in call signature for create_hierarchica…
prajwal-tech07 e7e3c03
refactor: use loguru logger.warning() and implement two-step mesh cre…
prajwal-tech07 ef478a2
refactor: update archetypes and tests to use mesh_layout parameter
prajwal-tech07 305e66a
test: update backward-compat tests to capture loguru warnings instead…
prajwal-tech07 b38fd64
Merge branch 'main' into feat/mesh-layout-rectilinear
prajwal-tech07 4372a3f
style: fix linting — run black, isort, remove empty f-strings
prajwal-tech07 4f48ed3
refactor: remove inline defaults and simplify m2m_connectivity_kwargs…
prajwal-tech07 0b6ff4a
refactor: rename mesh.kinds to mesh.connectivity, use **kwargs, updat…
prajwal-tech07 c7bdeac
Merge branch 'main' into feat/mesh-layout-rectilinear
leifdenby 51ea189
fix: pre-commit linting + notebook max_num_refinement_levels
prajwal-tech07 220fa44
Merge branch 'main' into feat/mesh-layout-rectilinear
leifdenby edbce82
refactor: separate coordinate creation (Step 1) from connectivity cre…
prajwal-tech07 eb4533f
refactor: move nx_mesh/ny_mesh computation into create_single_level_2…
prajwal-tech07 5335de6
refactor: move create_directed_mesh_graph to connectivity/directed an…
prajwal-tech07 b4f7dcf
refactor: rename connectivity/directed.py to general.py and clean up …
prajwal-tech07 106454e
fix: resolve lint failures and make plot tests windows-safe
prajwal-tech07 fae6188
fix: use NamedTemporaryFile handle directly in savefig per review sug…
prajwal-tech07 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.