Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add missing elements to VT JSON writer #363

Closed
ppebay opened this issue Mar 24, 2023 · 2 comments
Closed

Add missing elements to VT JSON writer #363

ppebay opened this issue Mar 24, 2023 · 2 comments
Assignees
@ppebay @nlslatt
Labels
enhancement New feature or request low-priority

Comments

@ppebay
Copy link
Contributor

ppebay commented Mar 24, 2023

This is a sequel to #354 , in particular to:

  • If there was other metadata in the input file, include it here too. The shared_node fields may influence how a load balancer behaves, and we would definitely want to document in the output file that the load balancer was run using those assumptions.
  • add communications (when available) and optional user-defined data
  • please feel free @nlslatt to add anything here
@ppebay ppebay added the enhancement New feature or request label Mar 24, 2023
@lifflander
Copy link
Contributor

This will be fixed as part of the VT-TV integration effort.

@cwschilly
Copy link
Contributor

Closed with #483

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ppebay ppebay

@nlslatt nlslatt

Labels
enhancement New feature or request low-priority
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@lifflander @ppebay @nlslatt @cwschilly