Skip to content

Conversation

@hsu000001
Copy link
Contributor

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).

@progval progval mentioned this pull request Dec 20, 2024
@hsu000001
Copy link
Contributor Author

Updated to delimit multiple fields in the mapping to a slash surrounded by spaces

@progval
Copy link
Member

progval commented Dec 21, 2024

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 aggregate.py?

@hsu000001
Copy link
Contributor Author

The definitions can be seen in the body of here:
https://github.com/DOI-USGS/modelcatalog-tam/blob/main/metadata_schema/BasicProfileSchema.py

There are some other definitions in additional files in that directory, e.g.,
https://github.com/DOI-USGS/modelcatalog-tam/blob/main/metadata_schema/PersonSchema.py

I have attempted to remove the changes from aggregate.py, I think it was only the crosswalk.csv, which I have reverted.

hsu000001 and others added 5 commits December 29, 2024 20:15
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)
@hsu000001
Copy link
Contributor Author

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.

@hsu000001
Copy link
Contributor Author

Am I able to merge this? Thanks.

@progval progval merged commit 97378ea into codemeta:master Feb 14, 2025
2 checks passed
tmorrell pushed a commit to caltechlibrary/codemeta that referenced this pull request Apr 11, 2025
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.

2 participants