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

Consider conformance of HybridSign static text #275

Open
j-d-b opened this issue Mar 21, 2022 · 4 comments
Open

Consider conformance of HybridSign static text #275

j-d-b opened this issue Mar 21, 2022 · 4 comments
Assignees
Labels
Cleanup This item relates to cleaning up the specification (e.g. removing deprecated items) Device Feed This item applies only to the WZDx Device Feed
Milestone

Comments

@j-d-b
Copy link
Collaborator

j-d-b commented Mar 21, 2022

Background

Currently, the static_sign_text property on the HybridSign object is optional. The property is described as:

The static text on the non-electronic component of the hybrid sign.

Issue

Members have asked several times why the static_sign_text is optional, and I couldn't find an answer in the specification documentation or my meeting notes. This issue seeks to start discussion that could lead to requiring this property in a future release.

Discussion

  • Discuss the reasoning for representing a HybridSign without providing static_sign_text.
  • Discuss if the static_sign_text property should be required.
@j-d-b j-d-b added Needs discussion This issue needs clarification/additional discussion or is inactive Smart Work Zones This issue/PR is related to the SWZ Subgroup Cleanup This item relates to cleaning up the specification (e.g. removing deprecated items) labels Mar 21, 2022
@j-d-b
Copy link
Collaborator Author

j-d-b commented Mar 21, 2022

For some initial thoughts:

Generally, WZDx tries to only make properties required that are critical to processing the data. For a hybrid sign, the only required property is dynamic_message_function, which is defined as:

The function the dynamic message displayed (e.g. a speed limit).

This can have a value of speed-limit, travel-time, or other. With speed-limit, the static_sign_text is not needed to know if what dynamic_message_text is used for. Perhaps this is why it is optional.

@sergebeaudry
Copy link

I agree that this could be required. I only see cases where some text exist on those. Speed Limit, Travel Time, height limit, width limit, etc.......
Even the dynamic_message_text might be required with potential to call it blanked

@j-d-b j-d-b self-assigned this Apr 1, 2022
@j-d-b j-d-b removed the Needs discussion This issue needs clarification/additional discussion or is inactive label Apr 21, 2022
@j-d-b
Copy link
Collaborator Author

j-d-b commented Apr 21, 2022

Based on SWZ subgroup co-chairs discussion, static_sign_text should be required. Thus the proposed change in the meantime is to indicate in the documentation that the property will be required in a future release.

This issue will remain open until v5.0, when the property can be required.

@tfoster-vm
Copy link

Could it stay open for being required in v5.0 and also be added still be added as optional until v5.0?

@j-d-b j-d-b added Device Feed This item applies only to the WZDx Device Feed and removed Smart Work Zones This issue/PR is related to the SWZ Subgroup labels Feb 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Cleanup This item relates to cleaning up the specification (e.g. removing deprecated items) Device Feed This item applies only to the WZDx Device Feed
Projects
None yet
Development

No branches or pull requests

3 participants