-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
DRAFT: Add Dockerfile #18
base: master
Are you sure you want to change the base?
Conversation
Hey! Great work! |
I noticed there is an issue where this configuration will not convert multiple files. This issue can be reproduced by setting the python convert.py /input /output --workers 10 --max 10 --min_length 10000 Then run |
Thank you for this, @gardner ! Is there any easy way to get this to work with or without a cuda device? I imagine it would need a different base image. Also, any reason to use 11.8 over 12 for cuda? |
Hi - I'm not an active contributor so please take what I say with a grain of salt. I would comment on an Issue if there was one. Please disregard if this isn't the place. In my particular use case, a Docker container that exposes an API I can repeated call over HTTP would be significantly more helpful and a Docker container used as a sort of command line function call. This makes it useful in a cloud deployment situation. E.g. I could just push this to Google Cloud Run and have a functioning/scaling endpoint to convert PDFs to markdown. Anyways, I'm going to be writing one for my own use but would be happy to post here when finished. |
@justinmclark, did you get around to doing this? Something like this would also be very useful for me right now! |
56bd9d6
to
10fa67e
Compare
Add Dockerfile
Update installation scripts to work without sudo if run as root
Set execution bit on *.sh