-
Notifications
You must be signed in to change notification settings - Fork 702
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
feat: update dependencies and remove constraint on pydantic #2841
Conversation
80464a8
to
7396c16
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updates look good, just had a question on the consistent-deps.sh
update
5ff9d85
to
89350d9
Compare
41b57c3
to
155794d
Compare
…ured-IO#2841) ### Description * The `consistent-deps.sh` was fixed to take into account the ingest dependencies, causing some errors to show up. New constriants were added to make that script pass. * Update all requirements without constraint on pydantic, allowing the latest version to be pulled in. * `pikepdf` is causing a conflict but there's a fix on their `main` branch, just need for the next release to be published. Opened up a question here to see if we can get that out any sooner: [Do releases happen on a schedule?](pikepdf/pikepdf#574). For now added `lxml<5` to the constraints. A couple optimizations: * `constraints.in` renamed to `constraints.txt` since the whole point is all dependencies are already pinned and the file never gets compiled * `constraints.txt` moved to a `requirements/deps` directory as this never gets compiled by `pip-compile` * Other dependency files updated to reference the new location of `base.in` and `constraints.txt` * make file updated since it was originally written to avoid the `base.in` and `constraints.in` file
…ured-IO#2841) ### Description * The `consistent-deps.sh` was fixed to take into account the ingest dependencies, causing some errors to show up. New constriants were added to make that script pass. * Update all requirements without constraint on pydantic, allowing the latest version to be pulled in. * `pikepdf` is causing a conflict but there's a fix on their `main` branch, just need for the next release to be published. Opened up a question here to see if we can get that out any sooner: [Do releases happen on a schedule?](pikepdf/pikepdf#574). For now added `lxml<5` to the constraints. A couple optimizations: * `constraints.in` renamed to `constraints.txt` since the whole point is all dependencies are already pinned and the file never gets compiled * `constraints.txt` moved to a `requirements/deps` directory as this never gets compiled by `pip-compile` * Other dependency files updated to reference the new location of `base.in` and `constraints.txt` * make file updated since it was originally written to avoid the `base.in` and `constraints.in` file
@@ -16,5 +16,5 @@ rapidfuzz | |||
backoff | |||
typing-extensions | |||
# NOTE(jennings): pinned due to later versions not supporting api_key_auth in UnstructuredClient | |||
unstructured-client>=0.15.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rbiseck3, do you happen to remember why this version was restricted? The newer parameters of the client aren't accessible with this restricted
Description
consistent-deps.sh
was fixed to take into account the ingest dependencies, causing some errors to show up. New constriants were added to make that script pass.pikepdf
is causing a conflict but there's a fix on theirmain
branch, just need for the next release to be published. Opened up a question here to see if we can get that out any sooner: Do releases happen on a schedule?. For now addedlxml<5
to the constraints.A couple optimizations:
constraints.in
renamed toconstraints.txt
since the whole point is all dependencies are already pinned and the file never gets compiledconstraints.txt
moved to arequirements/deps
directory as this never gets compiled bypip-compile
base.in
andconstraints.txt
base.in
andconstraints.in
file