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

fix break windows local and remote pipelines #2785

Merged
merged 3 commits into from
Aug 12, 2020
Merged

fix break windows local and remote pipelines #2785

merged 3 commits into from
Aug 12, 2020

Conversation

linbinskn
Copy link
Contributor

No description provided.

@@ -158,7 +158,7 @@ testCases:
configFile: test/config/examples/sklearn-regression.yml
setExperimentIdtoVar: $resumeExpId
# for subfolder in codedir test
launchCommand: mkdir -p ../examples/trials/sklearn/regression/subfolder && touch ../examples/trials/sklearn/regression/subfolder/subfile && nnictl create --config $configFile --debug
launchCommand: python3 -c "import os; os.mkdir('../examples/trials/sklearn/regression/subfolder'); open('../examples/trials/sklearn/regression/subfolder/subfile', 'a').close()" && nnictl create --config $configFile --debug
Copy link
Contributor

Choose a reason for hiding this comment

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

If the folder already exist, this line will throw exception

@QuanluZhang QuanluZhang requested a review from chicm-ms August 12, 2020 10:11
@ultmaster ultmaster merged commit f6991e8 into microsoft:master Aug 12, 2020
LovPe pushed a commit to LovPe/nni that referenced this pull request Aug 17, 2020
* fix break windows local and remote pipelines

* python os to separate platform

* Update integration_tests.yml

Co-authored-by: Yuge Zhang <Yuge.Zhang@microsoft.com>
(cherry picked from commit f6991e8)
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