Closed raissameyer closed 1 year ago
The review of the semantic mapping (following the update of the SSSOM guidance and @raissameyer's implementation thereof) is done above.
You'll notice that this was done in a line-by-line fashion as we think this should be the norm for a mapping file that links standards organisations: very thorough, with approvals explicit.
Thank you for the in depth review @pbuttigieg. I have addressed the comments as discussed, and updated the semantic and syntactic mapping file accordingly.
Please let me know if there are any further comments that should be addressed, otherwise I'll go ahead and also update the Support mapping file to reflect the changes in the semantic and syntactic mapping - which should be the final step in this PR.
This PR and issue in the SSSOM repo showed that - by following the old SSSOM guidance - we have sometimes used the SKOS predicates for narrow match and broad match incorrectly.
In issue #77 we reviewed the mapping predicates.
In this PR we will update our mapping files to fix this.
Updates for v2.0.0 of the mapping
Mapping predicates
predicate_id
updated toskos:narrowMatch
for the terms listed here: https://github.com/tdwg/gbwg/issues/77#issuecomment-995806725predicate_id
updated toskos:broadMatch
for the terms listed here: https://github.com/tdwg/gbwg/issues/77#issuecomment-995840816syntax_predicate_id
updated toskos:broadMatch
for the terms listed here: https://github.com/tdwg/gbwg/issues/77#issuecomment-997932073, https://github.com/tdwg/gbwg/issues/77#issuecomment-998646738Associated metadata
mapping_date
/syntax_mapping_date
andcreator_id
were updated to reflect the updates in the mapping predicatesAdditional metadata
confidence
/syntax_confidence
for the mapping between 0 and 1 (0 no confidence, 1 100% confidence; see The SSSOM 5-Star System For Mappings).Removed metadata
predicate_label
as there was no clear guidance on what those are for skos:narrowMatch and skos:broadMatch.