Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

Hotfix installation problem on Windows without conda/venv #2793

Merged
merged 1 commit into from
Aug 24, 2020

Conversation

liuzhe-lz
Copy link
Contributor

@liuzhe-lz liuzhe-lz commented Aug 14, 2020

On Windows site.getsitepackages()[0] can either be the base directory like C:\Python38, or site-packages. So all items in that list need to be checked.
The ideal solution is to find a reliable resource packaging machanism. data_files is not designed for this use case.

'''
for sitepackage in sitepackages:
for path in [Path(sitepackage).parents[2], Path(sitepackage)]:
if (path / 'nni' / 'main.js').is_file():
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is value of path / 'nni' / 'main.js'?

Copy link
Contributor Author

@liuzhe-lz liuzhe-lz Aug 24, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A Path object referring to main.js file in nni folder in path. It's expected to be C:\Python38\nni\main.js.

@ultmaster ultmaster merged commit da541bf into microsoft:v1.8 Aug 24, 2020
@liuzhe-lz liuzhe-lz mentioned this pull request Aug 26, 2020
@liuzhe-lz liuzhe-lz deleted the hotfix-install branch September 17, 2020 23:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants