You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
which causes no end of problems when people want to use Unstructured as a library, and also want to use other libraries at the same time. It also wastes a lot of bandwidth. This is particularly problematic when using Poetry.
And, if you use Poetry, as soon as someone caps the Python version, every Poetry project that uses it must also cap, even if you believe it is a detestable practice and confusing to users. It is also wrong unless you fully pin the dependency that forced the cap - if the dependency drops it in a patch release or something else you support, you no longer would need the cap. Even worse, if someone adds a cap or tightens a cap, unless they yank every single older release, a locking solver like Poetry or PDM will backsolve to the last versions without the cap so that the lock file it creates will be “valid” on all the Python versions you are requesting!
The text was updated successfully, but these errors were encountered:
In general the very hard version requirements in setup.py are quite bad and make it so that you basically can't use Unstructured in conjunction with other packages that might have slightly different requirements. For an application this kind of version pinning is a Good Thing. For a library it is a Bad Thing. Is Unstructured an application or a library?
It also leads to downloading gigabytes and gigabytes of CUDA junk because you have pinned the exact torch version and every torch version also pins its own specific versions of CUDA...
dhdaines
changed the title
Hard upper bound on Python version considered harmful
bug/Hard upper bound on Python version considered harmful
Dec 12, 2024
Unstructured's
setup.py
contains this line:which causes no end of problems when people want to use Unstructured as a library, and also want to use other libraries at the same time. It also wastes a lot of bandwidth. This is particularly problematic when using Poetry.
For why you should not do this: https://iscinumpy.dev/post/bound-version-constraints/#pinning-the-python-version-is-special
The text was updated successfully, but these errors were encountered: