-
Notifications
You must be signed in to change notification settings - Fork 93
add Usgsmodelcat mapping #370
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
Conversation
|
Updated to delimit multiple fields in the mapping to a slash surrounded by spaces |
|
Do you have a list of definitions for the USGS Model Catalog? I see the schema definition, but it doesn't explain the semantics of terms. And could you remove the changes to |
|
The definitions can be seen in the body of here: There are some other definitions in additional files in that directory, e.g., I have attempted to remove the changes from aggregate.py, I think it was only the crosswalk.csv, which I have reverted. |
author is an object, as is person Co-authored-by: Val Lorentz <[email protected]>
use only the URI Co-authored-by: Val Lorentz <[email protected]>
map url only to an official usgs website, other websites will be captured in weblinks Co-authored-by: Val Lorentz <[email protected]>
do not map hasSourceCode, used for different purpose Co-authored-by: Val Lorentz <[email protected]>
remove mapping to related_catalog_assets since it is unclear if it is a subset or superset (it could be another type of related asset)
|
I think that I just have addressed all of the comments or saw an acknowledgment of an explanation. I will await the next step, thanks. |
|
Am I able to merge this? Thanks. |
Initial mapping. Additional USGS Model Catalog data model details are at https://github.com/DOI-USGS/modelcatalog-tam
Unsure of how to map multiple relevant fields to one field (ex: code repository), and how to add conditional statements (ex: author).