-
Notifications
You must be signed in to change notification settings - Fork 7
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
TG2-VALIDATION_DEGREEOFESTABLISHMENT_STANDARD #275
Comments
Should be made CORE - see comments under #268 |
Updated notes to change "fail" to more explicit "This test must return NOT_COMPLIANT if there is leading or trailing whitespace or there are leading or trailing non-printing characters. " |
Source authority should be: bdq:sourceAuthority default = "Degree of Establishment Controlled Vocabulary List of Terms" {[https://dwc.tdwg.org/doe/]} {GBIF vocabulary API [https://api.gbif.org/v1/vocabularies/DegreeOfEstablishment/]} |
I think https://api.gbif.org/v1/vocabularies/DegreeOfEstablishment/concepts is OK as a source authority because it actually has an API, as long as it is understood that the actually vocabulary is maintained at https://dwc.tdwg.org/doe/ and the GBIF API is expected to remain up to date with that. |
Changed Source Authority from bdq:sourceAuthority default = "Darwin Core degreeOfEstablishment" {[https://dwc.tdwg.org/list/#dwc_degreeOfEstablishment]} {dwc:degreeOfEstablishment vocabulary API [https://api.gbif.org/v1/vocabularies/DegreeOfEstablishment/concepts]} to bdq:sourceAuthority default = "GBIF DegreeOfEstablishment Vocabulary" [https://api.gbif.org/v1/vocabularies/DegreeOfEstablishment]} {"dwc:degreeOfEstablishment vocabulary API" [https://api.gbif.org/v1/vocabularies/DegreeOfEstablishment/concepts]} |
Source Authority should be bdq:sourceAuthority default = "Degree of Establishment Controlled Vocabulary List of Terms" {[https://dwc.tdwg.org/doe/]} {GBIF vocabulary API [https://api.gbif.org/v1/vocabularies/DegreeOfEstablishment/concepts]} |
Changed Source Authority from bdq:sourceAuthority default = "GBIF DegreeOfEstablishment Vocabulary" [https://api.gbif.org/v1/vocabularies/DegreeOfEstablishment]} {"dwc:degreeOfEstablishment vocabulary API" [https://api.gbif.org/v1/vocabularies/DegreeOfEstablishment/concepts]} to bdq:sourceAuthority default = "Degree of Establishment Controlled Vocabulary List of Terms" {[https://dwc.tdwg.org/doe/]} {GBIF vocabulary API [https://api.gbif.org/v1/vocabularies/DegreeOfEstablishment/concepts]} |
We have to address a case missmatch between the TDWG vocabulary and the GBIF API. The TDWG vocabulary is authoriative, and has both Label and Controlled Value with an initial letter lower case, e.g. "native", the GBIF API has Name and Label.Value in mixed case, e.g. "Native". A lookup in the GBIF API has a cross reference to the TDWG vocabulary by IRI (thus only the GBIF API needs to be consulted to check dwciri values (though not consistently between vocabularies)), but not by label, implementors using the GBIF API would still have to consult the TDWG vocabulary to obtain the actual controlled value for the dwc: term. "native" should return Compliant, "Native" not compliant. referencing the GBIF API here does not assist either the specification or implementors. We should probably remove the GBIF API from the source authority for this set of VALIDATION_..STANDARD tests (but include it for the related AMENDMENT..._STANDARDIZED tests, as it provides alternative terms that can be used for standardization. This comment applies to #268, #269, #275, #276, #277, and #278 where the TDWG vocabulary has values with an initial lower case letter, and the GBIF API does not match, having values with an initial capital letter. Examples: Pathway: http://rs.tdwg.org/dwcpw/values/p039 Controlled Value = hullFouling Degree of Establishment: http://rs.tdwg.org/dwcdoe/values/d001 Controlled Value = native Establishment Means: http://rs.tdwg.org/dwcem/values/e001 Controlled Value = native |
I think the better path is to convince GBIF to correct their implementations of the vocabularies. It was probably an oversight or an attempt to follow a capitalization pattern that was established before these vocabularies were added. |
@tucotuco I agree. In the absence of that change, we should remove the reference to the GBIF API from the validations, and add a note for implementors to the ammendments. |
@tucotuco yes, ideal would be an alignment of the text strings in the API to the standard, though GBIF doesn't necessarily have to change the values of the Names in their API, they only need to provide the TDWG IRI and ControlledVocabulary value in consistent places where we can tell implementors to look for them. |
Thanks for the ping
It was indeed. Just for background, one design goal of the vocabulary server was to serve up various vocabularies (internal or external such as TDWG, IUCN etc) in a consistent manner through a unified API. When it comes to formats we're dealing with vocabularies that use lowerCamelCase, UpperCamelCase, lower_snake_case, CAPTIAL_SNAKE_CASE etc and - rightly or wrongly - the intent was to try and normalise things. That approach resulted in our following Concepts with UpperCamelCase and properties as lowerCamelCase similar to e.g. DwC at the time with Classes (HumanObservation). I'll chat with people here and comment back on the feasibility of changing at our end - it may have implications I'm not aware of (e.g. external API users). As I understand it the options are either we change the name to lowerCamelCase or add a field More to follow... |
On Thu, 18 Apr 2024 01:45:43 -0700 Tim Robertson ***@***.***> wrote:
As I understand it the options are either we change the name to
lowerCamelCase or add a field `controlledValue` - if I'm mistaken
please correct me.
Consistency within the API feels like a good thing.
Best path feels like it is to add a controlledValue term to the response. Given the presence of externalDefinitions, it might make sense to add structure to externalDefinitions, giving it the form:
key: 113
name: HullFouling
externalDefinitions:
[0]
iri: http://rs.tdwg.org/dwcpw/values/p039
controlledValue: hullFouling
|
Hi all
The GBIF concepts have been updated to lowerCamelCase to follow the TDWG convention for the following vocabularies: https://registry.gbif.org/vocabulary/Pathway/concepts |
GBIF vocabulary has now been aligned with Darwin Core. Thanks @timrobertson100 |
…, added to unit tests. Added implementations for core tests tdwg/bdq#284 tdwg/bdq#275 and tdwg/bdq#276 along with default methods and unit tests.
The text was updated successfully, but these errors were encountered: