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

Prepare for heroku-24 #2826

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

Prepare for heroku-24 #2826

wants to merge 3 commits into from

Conversation

jrochkind
Copy link
Contributor

@jrochkind jrochkind commented Dec 17, 2024

Minor tweaks necessary for moving to heroku-24 (ref #2688 ), as well as doc improvement encountered.

All these changes are backwards compat, so can be merged into master and deployed on heroku-22 as well, no problem.

To switch to heroku-24:

  1. Merge this
  2. Upgrade to heroku-24 in web UI or heroku stacks -- wont' take effect until next deploy
  3. Remove https://github.com/brandoncc/heroku-buildpack-vips from buildpacks, no longer necessary
  4. Deploy app
  5. On relevant app (eg production), need to re-set heroku config:set TESSDATA_PREFIX="/app/.apt/usr/share/tesseract-ocr/5/tessdata/" -r production

Note there may be a brief period after deploy but before you set TESSDATA_PREFIX where tesseract can't find non-English language modules. Shouldn't really be an issue.

This has been tested on staging, with heroku run "rspec system_env_spec" passing, plus a manual test of ingesting a TIFF and OCR'ing it. (Haven't manually tested everything, just that one, counting on tests!)

  • heroku-24 migration complete

@jrochkind
Copy link
Contributor Author

@jrochkind
Copy link
Contributor Author

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