Skip to content

feat: SU2 export#8

Open
bezmi wants to merge 1 commit into
mainfrom
feat_su2_export
Open

feat: SU2 export#8
bezmi wants to merge 1 commit into
mainfrom
feat_su2_export

Conversation

@bezmi

@bezmi bezmi commented Jun 13, 2026

Copy link
Copy Markdown
Owner

Another fable special. Adds SU2 export . Once fully reviewed, closes out #5. Depends on #6. This one is pretty big and might need to be split for effective code review.

@bezmi bezmi changed the base branch from main to feat_parametrised_geometry June 13, 2026 10:11
@bezmi bezmi mentioned this pull request Jun 13, 2026
@bezmi bezmi force-pushed the feat_parametrised_geometry branch from 890a907 to 7dddcfd Compare June 13, 2026 14:45
@bezmi bezmi force-pushed the feat_su2_export branch 2 times, most recently from b1689d0 to 06ca1b3 Compare June 14, 2026 13:19
@bezmi bezmi force-pushed the feat_parametrised_geometry branch from 7dddcfd to 21728f6 Compare June 17, 2026 06:10
@bezmi bezmi force-pushed the feat_su2_export branch from 06ca1b3 to 3244bc3 Compare June 17, 2026 06:10
@bezmi bezmi changed the base branch from feat_parametrised_geometry to main June 23, 2026 02:26
- egg/io/su2.py: export_su2() writing NDIME/NPOIN/NELEM/NMARK with
  VTK element types, deduplicated interface nodes, and automatic
  orientation fixing (CCW quads, positive-volume hexes)
- TopologyBuilder.tag_boundary() / BlockTopology.boundary_tags for
  named boundary markers, validated against shared interface faces
- blob example: --export flag plus blob/bottom/right/top/left tags
- tests/test_su2_export.py: 11 tests incl. round-trip parser

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@bezmi bezmi force-pushed the feat_su2_export branch from d8aca66 to 409fb68 Compare June 23, 2026 02:42
@bezmi bezmi changed the title feat: SU2 export, gdtk geometry module frontend, gdtk blob example feat: SU2 export Jun 23, 2026
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.

1 participant