Skip to content

Comments

Add Serialization Messaging (NGWPC-9482)#73

Draft
idtodd wants to merge 5 commits intodevelopmentfrom
idt-state-serialization
Draft

Add Serialization Messaging (NGWPC-9482)#73
idtodd wants to merge 5 commits intodevelopmentfrom
idt-state-serialization

Conversation

@idtodd
Copy link

@idtodd idtodd commented Feb 9, 2026

This PR adds messaging for creating, retrieving, and freeing serialization states for the T-Route BMI. The extent of the data saved includes data stored on the AbstractNetwork and DataAssimilation objects.

Note: Because of some structural changes to the BMI, this PR must be merged alongside the implementation of state saving in NGEN. Merging prior to the changes made there will break the communication of data between the two.

Additions

  • Messaging variables for working with state data.
  • Variables on the BMI representing the output results from a T-Route run.

Removals

Changes

Testing

  1. Test script demonstrating the new architecture can work with data loaded from NGEN output nexus CSVs.
  2. Testing on AWS Workspaces using the state saving branch of NGEN, the output results with the new architecture for sending data from NGEN to T-Route and interpreting that data.

Screenshots

Notes

Todos

Checklist

  • PR has an informative and human-readable title
  • Changes are limited to a single goal (no scope creep)
  • Code can be automatically merged (no conflicts)
  • Code follows project standards (link if applicable)
  • Passes all existing automated tests
  • Any change in functionality is tested
  • New functions are documented (with a description, list of inputs, and expected output)
  • Placeholder code is flagged / future todos are captured in comments
  • Visually tested in supported browsers and devices (see checklist below 👇)
  • Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
  • Reviewers requested with the Reviewers tool ➡️

Testing checklist

Target Environment support

  • Windows
  • Linux
  • Browser

Accessibility

  • Keyboard friendly
  • Screen reader friendly

Other

  • Is useable without CSS
  • Is useable without JS
  • Flexible from small to large screens
  • No linting errors or warnings
  • JavaScript tests are passing

@idtodd idtodd self-assigned this Feb 10, 2026
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