Skip to content

FDTD | Feature | Include IA infrastructure, Added XDMF/HDF5 movie outputs and frequency-domain volumetric slices - #413

Open
adrianarce-elemwave wants to merge 10 commits into
new-feature-observationfrom
feat/inlcude-ia-infrastructure
Open

FDTD | Feature | Include IA infrastructure, Added XDMF/HDF5 movie outputs and frequency-domain volumetric slices#413
adrianarce-elemwave wants to merge 10 commits into
new-feature-observationfrom
feat/inlcude-ia-infrastructure

Conversation

@adrianarce-elemwave

Copy link
Copy Markdown
Collaborator

Summary

Introduces an isolated, AI-ready development container and integrates the new observation-output pipeline.

What changed

  • Adds reproducible quality and dev Docker targets with compilers, HDF5, MPI, ParaView, Python tooling, and OpenCode support.
  • Configures Dev Container mounts, environment checks, VS Code debugging, CMake presets, and Fortran-focused OpenCode skills.
  • Adds the new output module behind SEMBA_FDTD_ENABLE_OUTPUT_MODULE.
  • Integrates output initialisation, updates, flushing, and shutdown into the solver time-stepping lifecycle.
  • Adds XDMF/HDF5 movie outputs and frequency-domain volumetric slices for electric, magnetic, and current-density fields.
  • Adds unit-test infrastructure and example input data for movie and frequency-slice probes.
  • Extends CI to build both legacy and new output-module configurations.

Validation

  • CI compiles the output module across GCC/Intel, MPI, MTLN, and precision configurations.
  • Python integration tests remain enabled for the legacy output path only.
  • Adds dedicated Fortran tests for output handling, VTK API, XDMF API, volumetric utilities, and test assertions.

Notes for reviewers

  • The new output pipeline is enabled by default with SEMBA_FDTD_ENABLE_OUTPUT_MODULE=ON. Next follow ups will remove legacy implementation alongside this flag
  • The existing observation implementation remains available by disabling that option.
  • This PR combines development-infrastructure changes with output-module integration.

@adrianarce-elemwave
adrianarce-elemwave force-pushed the feat/inlcude-ia-infrastructure branch from 5211aa1 to b08f053 Compare July 27, 2026 13:10
@adrianarce-elemwave
adrianarce-elemwave force-pushed the feat/inlcude-ia-infrastructure branch from b08f053 to f194b5e Compare July 28, 2026 08:05
@adrianarce-elemwave
adrianarce-elemwave force-pushed the feat/inlcude-ia-infrastructure branch from 73457b0 to 09509ee Compare July 28, 2026 14:18
@adrianarce-elemwave
adrianarce-elemwave marked this pull request as ready for review July 30, 2026 09:39
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