Skip to content

Remove most old example code#470

Draft
PhilipFackler wants to merge 1 commit into
developfrom
PhilipFackler/example-tests
Draft

Remove most old example code#470
PhilipFackler wants to merge 1 commit into
developfrom
PhilipFackler/example-tests

Conversation

@PhilipFackler

Copy link
Copy Markdown
Collaborator

Description

Remove example code that duplicates what the DynamicSimulation app does

Closes #390 (indirectly)

Proposed changes

  • Add existing "tiny" cases to CaseFormatTests to check JSON parser
  • Remove example source code
  • Create ref.csv files for cases that were still using the .hpp data
  • Create .solver.json files where needed
  • Update tests and input/output files to use same naming conventions

Checklist

  • All tests pass.
  • Code compiles cleanly with flags -Wall -Wpedantic -Wconversion -Wextra.
  • [N/A] The new code follows GridKit™ style guidelines.
  • [N/A] There are unit tests for the new code.
  • [N/A] The new code is documented.
  • The feature branch is rebased with respect to the target branch.
  • [N/A] I have updated CHANGELOG.md to reflect the changes in this PR. If this is a minor PR that is part of a larger fix already included in the file, state so.

@PhilipFackler PhilipFackler force-pushed the PhilipFackler/example-tests branch from fd2fdfe to 4f444a8 Compare June 26, 2026 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: mon.csv write race causes flaky parallel ctest failures

2 participants