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

Digital Specimen update to 0.2.0 #92

Merged
merged 2 commits into from
May 21, 2024
Merged

Digital Specimen update to 0.2.0 #92

merged 2 commits into from
May 21, 2024

Conversation

samleeflang
Copy link
Contributor

PR with a bunch of new fields.
Most are verbatim fields, which we now added to the datamodel.
We also added a couple of ID fields which weren't previously included.

We try to make this a new minor version upgrade of the 0 version, let's see how that works out.

Copy link
Contributor

@southeo southeo left a comment

Choose a reason for hiding this comment

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

🐞

"https://orcid.org/0000-0002-1825-0097"
]
},
"???:recordedByAgent": {
Copy link
Contributor

Choose a reason for hiding this comment

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

recordedByAgent is not in dwc so this needs to be ods term. how is this diff than recordedBy?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The difference is that recordedBy is a string object and this is an agent object. Still not completely sure where to put agents without overcomplicating things. We later added the generic agents array attribute so I will remove this specific one.

"ILL: Union Co. Wolf Lake by Powder Plant Bridge. 1 March 1975 Coll. S. Ketzler, S. Herbert\n\nMonotoma longicollis 4 ♂ Det TC McElrath 2018\n\nINHS Insect Collection 456782"
]
},
"materialEntity": {
Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Checked the classes in dwc and added the ones I could find. Not sure if it works that way through, as until now these were used as basisOfRecord

"ods:physicalSpecimenIdType",
"ods:sourceSystem",
"dcterms:license",
"???:institutionId"
Copy link
Contributor

Choose a reason for hiding this comment

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

this should be dwc:institutionID
https://dwc.tdwg.org/list/#dwc_institutionID

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

],
"$comment": "Is this necessary for specimens"
},
"???:protocolDescription": {
Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added

"description": "Unclear yet",
"$comment": "Unknown what this field should be"
},
"???:typeDesignatedBy": {
Copy link
Contributor

Choose a reason for hiding this comment

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

it was proposed here but no other references: dshorthouse/agents_actions#2

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We need to rework agents to a generic object, then this field will probably become obsolete

"After the recent rains the river is nearly at flood stage"
]
},
"???:collectorName": {
Copy link
Contributor

Choose a reason for hiding this comment

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

this is usually captured by recordedBy?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I wouldn't say recorder and collector are necessarily the same. However, just as with typeDesignatedBy, we need to rework the agents to a generic object which includes the relationship to the object.

Copy link
Contributor

@TomDijkema TomDijkema left a comment

Choose a reason for hiding this comment

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

Some useful terms added 👍

@samleeflang samleeflang requested a review from sharifX May 21, 2024 06:36
"examples": [
"location information not given for endangered species"
],
"$comment": "Feels like this field should be true or false and another field should contain the explanation"
Copy link
Contributor

Choose a reason for hiding this comment

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

yes, but the example shows as text desc. so let's use it with text for now.

"$ref": "https://schemas.dissco.tech/schemas/digitalobjects/0.2.0/digital-specimens/occurrences.json"
}
},
"???:EntityRelationship": {
Copy link
Contributor

Choose a reason for hiding this comment

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

should we mark this gbif:EntityRelationship to indicate that they are from the new data model?

"examples": [
"individuals"
],
"$comment": "Could this become a enum?"
Copy link
Contributor

Choose a reason for hiding this comment

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

we need to bring this up to dwc working group. I also see this in the desc "A dwc:organismQuantityType must have a corresponding dwc:organismQuantity. This term has an equivalent in the dwciri: namespace that allows only an IRI as a value, whereas this term allows for any string literal value."

@samleeflang samleeflang merged commit 90d6841 into master May 21, 2024
1 check passed
@samleeflang samleeflang deleted the feature/ds-0.2.0 branch May 21, 2024 07:24
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.

4 participants