Skip to content

Refactor semba fdtd launcher#372

Draft
lmdiazangulo wants to merge 2 commits intodevfrom
refactor-semba-fdtd
Draft

Refactor semba fdtd launcher#372
lmdiazangulo wants to merge 2 commits intodevfrom
refactor-semba-fdtd

Conversation

@lmdiazangulo
Copy link
Copy Markdown
Contributor

No description provided.

lmdiazangulo and others added 2 commits April 19, 2026 10:49
- Change nfde2sgg from type-bound subroutine to module-level function
  returning dtantesdecorregir (type(semba_fdtd_t) instead of class)
- Fix getcwd/chdir portability using IFPORT for Intel Fortran
- Add 3 new test cases: planewave_periodic, holland, sources_voltage
- Extract common test setup into helper module test_nfde2sgg_m
- Use EPSILON_VACUUM/MU_VACUUM constants instead of hardcoded values
- Add CFL violation check (dt should not increase after correction)
- All 118 tests pass with both GNU (gfortran) and Intel (ifx)

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