Skip to content

Refactor preprocess#373

Draft
lmdiazangulo wants to merge 2 commits intodevfrom
refactor-preprocess
Draft

Refactor preprocess#373
lmdiazangulo wants to merge 2 commits intodevfrom
refactor-preprocess

Conversation

@lmdiazangulo
Copy link
Copy Markdown
Contributor

No description provided.

lmdiazangulo and others added 2 commits April 19, 2026 14:43
…rate modules

Phase 1 of preprocess_geom.F90 refactoring:
- Move checkDielectricComponentTags, checkAnimatedComponentTags,
  checkLossyTags, searchtag to new preprocess_tags_m module
- Hoist getDifferentEdgeRatios and getDifferentFaceRatios from
  read_geomData internal contains to module-level (makes them testable)
- Add preprocess_tags_m.F90 to semba-main sources in CMakeLists.txt
- Zero behavior change: all call sites unchanged

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@lmdiazangulo lmdiazangulo added AI Mostly created with AI. Needs special review. Housecleaning Stuff related with style conventions, good practices, clean code, repo organization, etc. and removed AI Mostly created with AI. Needs special review. labels Apr 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI Mostly created with AI. Needs special review. Housecleaning Stuff related with style conventions, good practices, clean code, repo organization, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant