The following seems to have never made it into GitHub. Gobe reminded me of the second item in this list. I think most of these are handled already but I added this issue so that I can track a review to make certain that all of these all have been addressed either now or earlier.
Steve,
Please find, below, comments from the OGC Naming Authority regarding the draft GeoPose 1.0.
[x] Page 32: Requirement BQ-S-1 appears to have two different identifiers “/req/basic/quaternion-permissive/structure” and “/req/basic/quaternion/structure”.
[x] Page 33: Requirement SDU-4 appears to have two different identifiers “/req/sdu/orientation/quaternion” and “/req/sdu/orientation/quaternon”. Notice that an ‘i’ is missing from the latter.
[x] Section 7.3 has requirements with URIs that have uppercase letters (e.g. /req/Graph/structure and /req/Chain/structure). Please ensure that all characters in URIs of all requirements are lowercase.
[x] Annex A: Replace all /ats/ segments with /conf/
[x] Annex A has abstract tests with URIs that have uppercase letters (e.g. /ats/basic/Euler). Please ensure that all characters in URIs of all abstract tests are lowercase.
3DXScape
commented
2 years ago
The following seems to have never made it into GitHub. Gobe reminded me of the second item in this list. I think most of these are handled already but I added this issue so that I can track a review to make certain that all of these all have been addressed either now or earlier.
Steve,
Please find, below, comments from the OGC Naming Authority regarding the draft GeoPose 1.0.
[x] Section 5.1: Convention has been to make URIs of specification elements lowercase. This is not a requirement. Please consider changing “http://www.opengis.net/spec/GeoPose/1.0” to “http://www.opengis.net/spec/geopose/1.0”
[x] Page 32: Requirement BQ-S-1 appears to have two different identifiers “/req/basic/quaternion-permissive/structure” and “/req/basic/quaternion/structure”.
[x] Page 33: Requirement SDU-4 appears to have two different identifiers “/req/sdu/orientation/quaternion” and “/req/sdu/orientation/quaternon”. Notice that an ‘i’ is missing from the latter.
[x] Section 7.3 has requirements with URIs that have uppercase letters (e.g. /req/Graph/structure and /req/Chain/structure). Please ensure that all characters in URIs of all requirements are lowercase.
[x] Annex A: Replace all /ats/ segments with /conf/
[x] Annex A has abstract tests with URIs that have uppercase letters (e.g. /ats/basic/Euler). Please ensure that all characters in URIs of all abstract tests are lowercase.
Regards,
Gobe