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

ISSUE-282 Initial trial editing matrFacility with specific mining concepts #283

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

skybristol
Copy link

In response to the issue I posted for this, I'm throwing this in as a trial balloon. I want to see what is involved in contributing to SWEET via the process set up currently. For this area, I have a couple dozen additional mining facility type concepts pulled from the 1997 AGI resource indicated in prov:wasDerivedFrom.

I would specifically welcome comments on the following:

  • Is this the correct way to cite a source like this that has not been published in formal ontology form (at least not in a usable/citable way)?
  • What's the gold standard we are shooting for in terms of additional detail in annotations? There's more I could provide here within the established namespace properties.

@skybristol
Copy link
Author

I should add that I believe this simple addition of three mining facility concepts can be merged as is. I'll open another issue to complete work in this area once I understand the contribution pattern.

@skybristol skybristol changed the title Initial trial editing matrFacility with specific mining concepts ISSUE-282 Initial trial editing matrFacility with specific mining concepts Apr 25, 2023
Copy link
Collaborator

@smrgeoinfo smrgeoinfo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some updates for the new concept URIs. Also use dct:source to provide a biblio citation for the AGI glossary used.

@@ -147,6 +147,38 @@ somaf:Mine rdf:type owl:Class ;
rdfs:label "mine"@en .


### http://sweetontology.net/matrFacility/Adit
somaf:Mine rdf:type owl:Class ;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be somaf:Adit

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's not how we had this classified on our end, but I tend to agree. I was looking at the rest of this that comes from the USMIN source (with reference back to the AGI dictionary).



### http://sweetontology.net/matrFacility/BorrowPit
somaf:Mine rdf:type owl:Class ;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be somaf:BorrowPit



### http://sweetontology.net/matrFacility/GypsumQuarry
somaf:Mine rdf:type owl:Class ;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be somaf:GypsumQuarry



### http://sweetontology.net/matrFacility/Quarry
somaf:Mine rdf:type owl:Class ;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be somaf:Quarry

rdfs:subClassOf somaf:Quarry ;
rdfs:label "gypsum quarry"@en ;
skos:definition "An open or surface mineral working for the purpose of extracting gypsum"@en ;
prov:wasDerivedFrom "ISBN 0-922152-36-5" .
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

prov:wasDerivedFrom has range prov:Entity, not obviously compatible with a string literal. Seems to me using something like dct:source would make more sense in this context.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great! Also what I was looking for. I'll look across SWEET further for dct:source. I'm interested to see how we are citing sources elsewhere. That reference piece seems to be missing quite a bit right now.

Speaking of a lack of source info, I was looking at SWEET's instantiation of minerals today in relation to how I'm working to bring Mindat minerals, along with the Strunz and Dana classifications, into our knowledge graph. Seems like an area maybe SWEET should cede territory to a domain group, focusing on perhaps clarifying higher level classes that help domains de/re-contextualize?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree, SWEET should just bridge to a mineral domain ontology (didn't @smrgeoinfo create one?) as per @skybristol 's comment above.

@skybristol
Copy link
Author

@smrgeoinfo - Thanks so much for your response. I really appreciate the input. I'll update per your suggestions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants