Skip to content

Commit 4262cf8

Browse files
otajkrshrimali
authored andcommitted
Use valid pypi versions for install in assistant (#14750)
Co-authored-by: Kushashwa Ravi Shrimali <[email protected]>
1 parent e72e5b9 commit 4262cf8

File tree

1 file changed

+2
-6
lines changed

1 file changed

+2
-6
lines changed

.actions/assistant.py

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
import datetime
22
import glob
33
import json
4-
import logging
54
import os
65
import re
76
import shutil
@@ -17,7 +16,6 @@
1716

1817
import fire
1918
import pkg_resources
20-
from packaging.version import parse as version_parse
2119

2220
REQUIREMENT_FILES = {
2321
"pytorch": (
@@ -124,10 +122,8 @@ def download_package(package: str, folder: str = ".", version: Optional[str] = N
124122
url = f"https://pypi.org/pypi/{PACKAGE_MAPPING[package]}/json"
125123
data = json.load(urlopen(Request(url)))
126124
if not version:
127-
versions = list(data["releases"].keys())
128-
versions = sorted(versions, key=lambda x: version_parse(x))
129-
logging.debug(f"Available versions: {versions}")
130-
version = versions[-1]
125+
pypi_vers = pypi_versions(PACKAGE_MAPPING[package])
126+
version = pypi_vers[-1]
131127
releases = list(filter(lambda r: r["packagetype"] == "sdist", data["releases"][version]))
132128
assert releases, f"Missing 'sdist' for this package/version aka {package}/{version}"
133129
release = releases[0]

0 commit comments

Comments
 (0)