Skip to content

Conversation

@mawelborn
Copy link
Contributor

  • Conditionally sets all three prediction text attributes when the text has been changed.
    • prediction["normalized"]["formatted"]
    • prediction["normalized"]["text"]
    • prediction["text"]
    • Previously only prediction["normalized"]["formatted"] and prediction["text"] were set, and unconditionally. Which at current time, avoids edge cases in review. But this future proofs it a little more, and avoids overwriting the raw OCR text necessarily when the text hasn't been changed.
  • More robustly sets prediction["normalized"]["structured"]["checked"] and
    prediction["normalized"]["structured"]["signed"] for form extractions.
  • Uses the more idiomatic name utils in place of utilities.
  • Configures Poe as a plugin rather than as a dev dependency, which means it can be invoked with the shorter command
    poetry poe <task> in addition to the original poetry run poe <task>.

@mawelborn mawelborn merged commit 98cb205 into main Mar 20, 2025
6 checks passed
@mawelborn mawelborn deleted the mawelborn/poetry-improvements branch March 20, 2025 21:36
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.

2 participants