search

monarch-initiative / dipper

Data Ingestion Pipeline for Monarch
https://dipper.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
57 stars 26 forks source link

Rename ClinVar module and update docs #823

Closed cmungall closed 5 years ago

cmungall commented 5 years ago

Trying to figure out clinvar ingest from dipper docs.

It points to:

https://dipper.readthedocs.io/en/latest/dipper.sources.ClinVarXML_alpha.html

Generally not a good idea to embed release state in the file/module name.

Docs say:

_clinvarxmlalpha First pass at converting ClinVar XML into RDF triples to be ingested by SciGraph. These triples conform to the core of the SEPIO Evidence & Provenance model 2016 Apr

this doesn't look so good if we are pointing people at our docs to figure out our modeling

TomConlin commented 5 years ago

Generally not a good idea to embed release state in the file/module name.

https://github.com/monarch-initiative/dipper/pull/819/files#r317832863