ClearML/clearml-agent
1
0
Fork 0
mirror of https://github.com/clearml/clearml-agent synced 2025-02-07 13:26:08 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix pre-installed packages are ignored when installing a git package wheel. Reinstalling a git+http link is enough to make sure all requirements are met/installed (trains issue #196)

Browse Source
This commit is contained in:
allegroai 2020-09-05 16:34:36 +03:00
parent 121dec2a62
commit ca89a1e322
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
trains_agent/helper/package/external_req.py
View File

@ -67,7 +67,9 @@ class ExternalRequirements(SimpleSubstitution):
self.post_install_req_lookup[package_name[0]] = req.req.line self.post_install_req_lookup[package_name[0]] = req.req.line
except: except:
pass pass
if not PackageManager.out_of_scope_install_package(req_line, "--ignore-installed"): # no need to force reinstall, pip will always rebuilt if the package comes from git
# and make sure the required packages are installed (if they are not it will install them)
if not PackageManager.out_of_scope_install_package(req_line):
raise ValueError("Failed installing GIT/HTTPs package \'{}\'".format(req_line)) raise ValueError("Failed installing GIT/HTTPs package \'{}\'".format(req_line))
def replace(self, req): def replace(self, req):