-
-
Notifications
You must be signed in to change notification settings - Fork 16
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
Salvage OCR / Barcode Scanning code for future optimization #329
Comments
I accidentally blew up the important branch for this because it was really old and just cluttering up our branches... whoops. Good news is I still had a local version of the branch. This is the most important file for OCR-related work, I believe.
|
I think this file is less important, but sharing anyway.
|
One last one. This was converted to a txt file - I think maybe because it was old or we didn't want it to be picked up by Airflow.
|
Marking as complete. This was prework for related issue #328. The key pieces of information are in my most recent comments. |
Problem Statement
Right now, @lprzychodzien has some great starter code for doing OCR / Barcode scanning in one of the two branches below:
We want to pull the good stuff from those branches and stick them into perhaps a separate python script (maybe
ocr.py
) in the airflow/dags/dailymed folder so that we can save it for later, reference it in the #328 issue, and then delete the above branches to avoid confusion and reduce our branch count.Criteria for Success
All the good OCR / Barcode Scanning is safely stored for future work in the main branch and we don't have any more dailymed branches.
Additional Information
Make note of dependencies needed in the python code - don't actually add the dependencies to requirements.txt for this PR.
The text was updated successfully, but these errors were encountered: