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

updating sssom file with metadata #162

Closed
wants to merge 4 commits into from
Closed

Conversation

diatomsRcool
Copy link
Contributor

Added metadata to the sssom file:
mapping_tool
mapping_date
creator_id
license

Copy link
Contributor

@matentzn matentzn left a comment

Choose a reason for hiding this comment

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

You dont need the license column, as you have license already in the header!

@matentzn
Copy link
Contributor

this sssom files seems tiny.. There should be thousands of matches in there?

@diatomsRcool
Copy link
Contributor Author

This was done manually by me based on the mappings that Bill did. I hear that a conversion from boomer output to SSSOM is in the works. In the meantime, I just grabbed the exact matches and this was all I could find. Maybe there is another file somewhere?

@matentzn
Copy link
Contributor

If these are manual mappings, that seems fine. I don't understand the ECTO mapping framework well enough, but my sense is that we need some kind of overarching ECTO mapping file that includes all mappings to external ontologies - which should be in the higher hundreds..

@diatomsRcool
Copy link
Contributor Author

another issue is that this was run when ECTO was very small. Prob needs to be run again now that there are way more classes.

@diatomsRcool
Copy link
Contributor Author

If you are happy with the removal of the licensing column, I can go ahead and merge this PR

@diatomsRcool
Copy link
Contributor Author

addresses issue #117

@matentzn
Copy link
Contributor

Need some time to think about this one, not sure how it's related to the automated mappings

@matentzn
Copy link
Contributor

Hey @diatomsRcool I believe we can remove this file now, as I think its redundant with https://github.com/EnvironmentOntology/environmental-exposure-ontology/blob/master/mappings/ecto.sssom.tsv

Was this file here originally created automatically? its a bit odd that some stuff that was matched here is not anymore matched in the new run, but maybe there are good explanations for this. Where did the ecto-exact-001.tsv orginially come from? If its automated pipeline, you can close this pull request (without merging) and delete the file.

@diatomsRcool
Copy link
Contributor Author

diatomsRcool commented Apr 23, 2021

The ecto-exact-001.tsv was literally made my me with my own 10 fingers on the keyboard. I just wanted to get something in sssom format. I will close this PR.

@matentzn
Copy link
Contributor

Hmm, maybe we need this in the end.. I will move it somewhere else.

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