-
Notifications
You must be signed in to change notification settings - Fork 10
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
warp CTC building is failed #35
Comments
@ysk24ok Could you check this please for CPU tag also. |
It seems pip 20.3 doesn't allow mismatch between version in METADATA ( When I apply the following patch to match the version and try to upload the wheel to testpypi.org , diff --git a/pytorch_binding/wheel/rename_wheels.py b/pytorch_binding/wheel/rename_wheels.py
index d6072a5..a08a810 100644
:
return out.decode('utf-8').split()[-2][:-1].replace('.', '')
+def get_torch_version():
+ major_ver, minor_ver = torch.__version__.split('.')[:2]
+ return major_ver + minor_ver
+
+
+def get_local_version_identifier(enable_gpu):
+ local_version_identifier = '+torch{}'.format(get_torch_version())
+ if enable_gpu:
+ local_version_identifier += ".cuda{}".format(get_cuda_version())
+ else:
+ local_version_identifier += ".cpu"
+ return local_version_identifier
+
+
if torch.cuda.is_available() or "CUDA_HOME" in os.environ:
enable_gpu = True
# For CUDA10.1, libcublas-10-2 is installed
@@ -75,9 +89,10 @@ ext_modules = [
)
]
+base_version = "88.77.66"
setup(
name="warpctc_pytorch",
- version="0.2.1",
+ version=base_version + get_local_version_identifier(enable_gpu),
description="Pytorch Bindings for warp-ctc maintained by ESPnet",
url="https://github.com/espnet/warp-ctc",
author=','.join([ the following error occurs. $ twine upload -r testpypi dist/warpctc_pytorch-88.77.66+torch16.cuda102-cp38-cp38-manylinux1_x86_64.whl
Uploading distributions to https://test.pypi.org/legacy/
Enter your username: espnet
/opt/pyenv/versions/3.8.5/lib/python3.8/site-packages/twine/auth.py:72: UserWarning: No recommended backend was available. Install a recommended 3rd party backend package; or, install the keyrings.alt package if you want to use the non-recommended backends. See https://pypi.org/project/keyring for details.
warnings.warn(str(exc))
Enter your password:
Uploading warpctc_pytorch-88.77.66+torch16.cuda102-cp38-cp38-manylinux1_x86_64.whl
100%|█████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 3.00M/3.00M [00:02<00:00, 1.55MB/s]
NOTE: Try --verbose to see response content.
HTTPError: 400 Bad Request from https://test.pypi.org/legacy/
'88.77.66+torch16.cuda102' is an invalid value for Version. Error: Can't use PEP 440 local versions. See https://packaging.python.org/specifications/core-metadata for more information. The error is raised here in wheelhouse. The solution is:
|
Oh, I found an another solution,
|
I have uploaded [root@545fd607d18e pytorch_binding]# pip3 --version
pip 20.3.1 from /opt/pyenv/versions/3.8.5/lib/python3.8/site-packages/pip (python 3.8)
[root@545fd607d18e pytorch_binding]# pip3 install warpctc_pytorch==88.77.66+torch16.cuda102 -f https://github.com/ysk24ok/wheel_serving_test/blob/main/warpctc_pytorch-88.77.66+torch16.cuda102-cp38-cp38-manylinux1_x86_64.whl?raw=true
Looking in links: https://github.com/ysk24ok/wheel_serving_test/blob/main/warpctc_pytorch-88.77.66+torch16.cuda102-cp38-cp38-manylinux1_x86_64.whl?raw=true
WARNING: Skipping page https://github.com/ysk24ok/wheel_serving_test/blob/main/warpctc_pytorch-88.77.66+torch16.cuda102-cp38-cp38-manylinux1_x86_64.whl?raw=true because the HEAD request got Content-Type: application/octet-stream.The only supported Content-Type is text/html
Collecting warpctc_pytorch==88.77.66+torch16.cuda102
Using cached https://github.com/ysk24ok/wheel_serving_test/blob/main/warpctc_pytorch-88.77.66+torch16.cuda102-cp38-cp38-manylinux1_x86_64.whl?raw=true (3.1 MB)
Installing collected packages: warpctc-pytorch
Successfully installed warpctc-pytorch-88.77.66+torch16.cuda102 It's a hassle for users to specify the full URL for If there is no objection, I'll proceed in this way. |
That sounds like the best solution for now. |
Until pip 21 is released, we can use the old resolver by adding
|
That sounds good. Thanks, @chintu619! |
@chintu619 Thanks for sharing. |
@ysk24ok, warp CTC building is failed (possibly due to pep440 introduced in pip20.3???)
https://github.com/espnet/espnet/runs/1473939967
Could you check this?
The text was updated successfully, but these errors were encountered: