Skip to content

Add Simulation Data Serialization docs page for VTK export#363

Draft
Copilot wants to merge 6 commits into
mainfrom
copilot/add-simulation-data-serialization
Draft

Add Simulation Data Serialization docs page for VTK export#363
Copilot wants to merge 6 commits into
mainfrom
copilot/add-simulation-data-serialization

Conversation

Copilot AI commented Apr 17, 2026

Copy link
Copy Markdown
Contributor
  • Inspect current VTK serialization docs and current vector-field output
  • Check repository test baseline (core tests currently fail before changes because the main environment is not instantiated: ACSets missing)
  • Consult VTK/ParaView XML documentation and WriteVTK behavior for vector arrays
  • Update the VTK docs example so the point-data section marks the flux array as the active vector field for ParaView
  • Verify the generated .vtu XML contains both NumberOfComponents="3" on the flux DataArray and Vectors="flux" on PointData
  • Re-run a focused local check for the serialization example
  • Run parallel validation

Copilot AI linked an issue Apr 17, 2026 that may be closed by this pull request
…with WriteVTK.jl

Agent-Logs-Url: https://github.com/AlgebraicJulia/Decapodes.jl/sessions/693025a5-9466-4209-b65d-1e97c1aa1a48

Co-authored-by: lukem12345 <70283489+lukem12345@users.noreply.github.com>
Copilot AI changed the title [WIP] Add support for writing simulation data to VTK Add Simulation Data Serialization docs page for VTK export Apr 17, 2026
Copilot AI requested a review from lukem12345 April 17, 2026 21:27
@lukem12345

Copy link
Copy Markdown
Member

@jpfairbanks If you have paraview installed locally still you could check this out

@lukem12345

Copy link
Copy Markdown
Member

Checks out:

Screen.Recording.2026-04-18.at.2.14.55.PM.mov

@lukem12345

Copy link
Copy Markdown
Member

The vector field was not immediately viewable in paraview, but that may be user error. The X Y and Z variables are all viewable.

@jpfairbanks

Copy link
Copy Markdown
Member

Assuming the tests and docs build, this LGTM

@lukem12345

lukem12345 commented Apr 22, 2026

Copy link
Copy Markdown
Member

I would want to hold off on merging until someone can check that the vector field data (the flux variable) is serialized appropriately.

paraview_data.zip

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.

Support writing simulation data to VTK

3 participants