-
Notifications
You must be signed in to change notification settings - Fork 0
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
Converter
exporters
#8
Comments
Fantastic idea! THANK YOU! |
This topic came up when working on monarch-initiative/mondo-ingest#394. Nico suggestion on how to do this:
|
@joeflack4 can you point me towards the code that saves these CSVs? maybe worth having an I/O function upstream in |
@cthoyt I appreciate you chiming in! I wrote in the OP that "unless Charlie thinks it's a good idea for this functionality to go in the curies package", but I think actually we would prefer that. I don't foresee SemanticSQL having the bandwidth to add EPMs now but I could be wrong. Here's an example of a Actually though I just realized I am not sure if the headers ( |
Overview
mondo-ingest
utilizes SemanticSQL, which requires prefix maps in CSV form. We should create our own function inmondolib
(unless Charlie thinks it's a good idea for this functionality to go in thecuries
package) that can generate this CSV so that we have 1 less place to maintain mappings.Sub-tasks
prefixes.csv
(Exporters to support Monarch use cases biopragmatics/curies#106)robot --add-prefixes
format:context.json
(Exporters to support Monarch use cases biopragmatics/curies#106) (docs)docs/developer/add-new-source.md
: Remove sections about the above 2 files needing to be statically updated.Additional information
Context
Comments in the synchronization: subclass axioms PR: 1, 2
Possible design
I think we should maintain these static files:
metadata/mondo.sssom.config.yml
metadata/SOURCE.yml
for each sourceThen, we should have some means of reading these in and instantiating a
curies.Converter
and from that export to the needed formats.Related
Converter
exporters mondo-ingest#371The text was updated successfully, but these errors were encountered: