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.

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

Revisit if using both SFSS and SFVSC is necessary #1117

Open
cstirdivant opened this issue Aug 2, 2023 · 1 comment
Open

Revisit if using both SFSS and SFVSC is necessary #1117

cstirdivant opened this issue Aug 2, 2023 · 1 comment
Assignees
Labels
ontology model Extensions or edits to the ontology model

Comments

@cstirdivant
Copy link
Collaborator

SFVSC (and other speed control types) require the run status and command field, therefore including SFSS is extraneous.

We should either:

  • Remove run_command and run_status as required on the VSC types
  • Revise precedent so that if a canonical type uses a VSC type it does not need the SS type (explore if this has ramifications for downstream analytics)
@shambergoldstein
Copy link
Collaborator

Also revisit if it makes sense to have speed_mode in the VSC types

@cstirdivant cstirdivant self-assigned this Aug 15, 2023
@cstirdivant cstirdivant added the ontology model Extensions or edits to the ontology model label Feb 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ontology model Extensions or edits to the ontology model
Projects
None yet
Development

No branches or pull requests

2 participants