Skip to content
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

Add functionality to the transcribe_documents management command #25

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Conversation

DanielSwain
Copy link
Contributor

Add --slice, --dry-run, --verbosity arguments

options parameter is required but no value is necessary when saving a document
Output is determined by --verbosity:  only output the transcribed text if verbosity is 3.
Order documents by title to ensure iterating in the same order each time.
When verbosity is >= 1, output the document number and the slice information.
Output count of documents that will be transcribed and blank lines for readability.
Only output statement about falling back to tesseract if verbosity >= 2.
Only output transcribed text if verbosity >= 3.
Arguments added to the transcribe_documents management command.
@codecov
Copy link

codecov bot commented Apr 16, 2019

Codecov Report

Merging #25 into master will decrease coverage by 3.93%.
The diff coverage is 62.50%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #25      +/-   ##
==========================================
- Coverage   89.72%   85.79%   -3.94%     
==========================================
  Files          15       15              
  Lines         146      176      +30     
==========================================
+ Hits          131      151      +20     
- Misses         15       25      +10     
Impacted Files Coverage Δ
...xtract/management/commands/transcribe_documents.py 66.66% <61.29%> (-33.34%) ⬇️
src/wagtail_textract/handlers.py 86.95% <62.50%> (+10.76%) ⬆️
src/wagtail_textract/models.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 645b454...d378fcf. Read the comment docs.

@DanielSwain
Copy link
Contributor Author

@khink Looks like this is only failing because of lack of tests, is that correct? I'll try to get tests completed soon.

>=1.11,<2.0 is the current Wagtail requirement for six
Update requirement for six library:  >=1.11,<2.0
* 'master' of https://github.com/fourdigits/wagtail_textract:
  Back to development: 1.2
  Preparing release 1.1
  Update README: Contributors
  Update changelog
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.

1 participant