Skip to content

Conversation

@jrlegrand
Copy link
Member

Fixes #1

What did you change?

Removed everything aside from the core parsing functionality. This includes:

  • API w/ logic for feedback loop - Rationale for removing this is that more than likely the thing we will use this for in the future is training a ML model or helping a LLM. The original intent for this API / feedback loop was for users / pharmacists giving feedback. Times have changed. This is just bulky and unnecessary for this repo at this point.
  • MySQL database setup
  • Gunicorn hosting setup
  • Frontend website - This was more or less my frontend marketing homepage bundled up with all of the other things above. It did have some sort of basic login ability for reviewers to log in, but again, this is bulky and unnecessary at this point.

Re-organize folder structure.

Add main.py which imports parser functionality and has CLI arguments for:

  1. Parsing a single sig and returning JSON in the terminal
  2. Parsing a CSV file of sigs and outputting the results in a separate CSV file

Rationale

For a long time, have been wanting to remove the bloated extra parts of this repo and make it super easy for someone to clone the repo and just write a simple command in the terminal to evaluate the accuracy of the parser - or to quickly use it to parse a full CSV. The functionality has always been there, but was obscured by the unnecessary scaffolding of the past focus of this project which was to kind of make a business out of it. That ship has sailed and I've since open sourced this all with the hopes that it can enhance LLMs or ML models to become the best open-source sig parser.

Testing

@jrlegrand jrlegrand requested a review from Kingstead September 28, 2024 18:39
@jrlegrand jrlegrand self-assigned this Sep 28, 2024
@jrlegrand jrlegrand merged commit 48ee092 into main Sep 29, 2024
@jrlegrand jrlegrand deleted the parserx-package branch September 30, 2024 04:18
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.

Turn ParseRx into a python module with CLI arguments

3 participants