-
Notifications
You must be signed in to change notification settings - Fork 49
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
Mismatch Between python_requires and requirements.txt Constraints #122
Comments
Hey Gordon, if you are having troubles on installation, the below is the readme for installation quick start, it worked for me let me know if you have any problems!
|
Thanks for your answer, I got it to install with python version 3.10., I wouldn't want to use an ancient version like 3.7, I mean even 3.8. is deprecated now in most packages. Had to ship around some problems with YOLOX first though: I still think the issue is valid, since the main problem stated here is, that the setup files for this YOLOV are not valid, i.e. some of the fixed requirements are not available for newer python versions, but the setup.py states that anything above python 3.6. will work. |
Hi @gboeer if i understand this thread, you got to installing with newer python versions by making the changes to the YOLOX github repo, as pointed in the issue you raised? but i'm not sure how it translated to enabling this installation. can you please explain more. secondly, i'm using a M-series Mac and it seems I have to uninstall opencv-python and use opencv-python-headless as i keep getting an error from the former, if you use a similar device, did you get similar challenges? i appreciate your response on these issues. |
Hi @onyekaokonji, In short: The ticket here is merely about the incorrect setup.py and requirements.txt contained in this repo, which won't work for newer python versions. I am working in a headless server using docker images so I can't say anything about Mac related issues. More often than not I will get an error like: In those cases, it usually helps to do |
thank you @gboeer i eventually got it to install and run on Python 3.12 by removing the version specific numbers on onnx_runtime and trusting that the latest version will have some backward compatibility with the version they used in the repo. |
Hi,
I believe there’s a potential mismatch between the
python_requires
specification insetup.py
and the fixed package version specified inrequirements.txt
.Here are the details:
In setup.py, the package specifies:
In requirements.txt, the following is specified:
However, when using Python 3.10.15 (in my case):
The lowest installable version of onnxruntime via pip install is 1.12.0, not 1.8.0.
This suggests that onnxruntime==1.8.0 is incompatible with Python 3.10. As a result, it becomes difficult to use this package as is on Python versions greater than 3.9, even though the python_requires specification claims support for Python versions >= 3.6.
Expected Behavior:
The python_requires and requirements.txt should align so that the package can be installed with Python versions that are officially supported.
Suggested Fix:
Update requirements.txt to include a compatible version range for onnxruntime that works with Python 3.10.
Alternatively, adjust python_requires in setup.py to reflect the actual supported Python versions based on dependency compatibility.
Additional Context:
This mismatch can create confusion for users who are trying to use the package on newer Python versions. Please let me know if I’m misunderstanding anything or if there are other considerations.
The text was updated successfully, but these errors were encountered: