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

EDR V1.2 OAB Review, Comment 2: Group commmon requirements #597

Open
chris-little opened this issue Feb 19, 2025 · 2 comments
Open

EDR V1.2 OAB Review, Comment 2: Group commmon requirements #597

chris-little opened this issue Feb 19, 2025 · 2 comments
Labels
API EDR V1.2 Non-breaking change for Version 1.2

Comments

@chris-little
Copy link
Contributor

From @cnreediii The following requirements in the Class Queries

Requirement A.24: /req/queries/position
Requirement A.25: /req/edr/rc-area
Requirement A.26: /req/edr/rc-cube
Requirement A.27: /req/edr/rc-trajectory
Requirement A.28: /req/edr/rc-corridor
Requirement A.29: /req/queries/radius
Requirement A.31: /req/edr/rc-locations

Have a common set of SHALLs, MAYs, and SHOULDs. Could the common characteristics be specified in a single requirement instance and then additional requirements add specifics for each query type? This would really shorten that section.

@chris-little chris-little changed the title EDR V1.2 OAB Review, Comment 2 Group commmon requirements EDR V1.2 OAB Review, Comment 2: Group commmon requirements Feb 19, 2025
@chris-little
Copy link
Contributor Author

We will discuss this in the EDR API SWG, but I am minded to stay with the verbose but flat approach. We considered similar approaches in other parts of the spec, such as common shared attributes, and the hierarchical approach sometimes makes the text harder to understand.

@chris-little chris-little added the API EDR V1.2 Non-breaking change for Version 1.2 label Feb 19, 2025
@cnreediii
Copy link

@chris-little Thanks. Will abide by what the SWG decides :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API EDR V1.2 Non-breaking change for Version 1.2
Projects
None yet
Development

No branches or pull requests

2 participants