ClearML/clearml
1
0
Fork 0
mirror of https://github.com/clearml/clearml synced 2025-04-09 15:15:20 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix git+https requirements handling, resolve top_level.txt package name (kerastuner from git was not detected)

Browse Source
This commit is contained in:
allegroai 2020-08-18 11:51:04 +03:00
parent 8112b6b44f
commit df0e5deb02
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
trains/utilities/pigar/reqs.py
View File

@ -363,6 +363,7 @@ def _search_path(path):
elif version.endswith('egg'):
version = version.rsplit('.', 1)[0]
mapping_pkg_name = pkg_name
# pep610 support. add support for new pip>=20.1 git reference feature
git_direct_json = os.path.join(path, file, 'direct_url.json')
if os.path.isfile(git_direct_json):
@ -376,14 +377,13 @@ def _search_path(path):
# Bugfix: package name should be the URL link, because we need it unique
# mapping[pkg_name] = ('-e', git_url)
mapping[pkg_name] = ('-e {}'.format(git_url), '')
pkg_name, version = '-e {}'.format(git_url), ''
continue
except Exception:
pass
# default
mapping[pkg_name] = (pkg_name, version)
mapping[mapping_pkg_name] = (pkg_name, version)
# analyze 'top_level.txt' if it exists
top_level = os.path.join(path, file, 'top_level.txt')