search

callahantiff / OMOP2OBO

OMOP2OBO: A Python Library for mapping OMOP standardized clinical terminologies to Open Biomedical Ontologies
http://tiffanycallahan.com/OMOP2OBO_Dashboard
MIT License
83 stars 12 forks source link

Move main.py into package and add vanity CLI #57

Open cthoyt opened 3 years ago

cthoyt commented 3 years ago

Closes #56

This PR does the following:

  1. Moves main.py into the main package hierarchy. My personal preference is to name the file cli.py, but I'll leave this as an option to you
  2. Adds a vanity CLI using python entrypings (see changes in setup.py)
  3. Updates documentation in README.rst and jupyter notebook
  4. Bonus: add __main__.py for usage as python -m omop2obo as well as docs in the top explaining what's going on

Happy ontologizing!

callahantiff commented 3 years ago

@cthoyt - Thanks so much for doing this. I've got some big deadlines this week and will work on merging your changes this weekend.

cthoyt commented 3 years ago

@callahantiff no rush! good luck

cthoyt commented 2 years ago

Greetings from the future @callahantiff

congrats on the 1.0 release, I just updated the code vs the main branch and this could still be merged if you want

callahantiff commented 2 years ago

Thanks so much, @cthoyt!! - I am so sorry that I have not included the awesome work you did in release v1.0.0. I had to rush this release and it needed to follow a specific format for a downstream application. I am currently working on v2.0.0 and am planning on including the awesome change you made in that. I hope that's OK!