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

Remove paiYarn mode #3327

Merged
merged 40 commits into from
Jan 26, 2021
Merged

Remove paiYarn mode #3327

merged 40 commits into from
Jan 26, 2021

Conversation

SparkSnail
Copy link
Contributor

  1. Remove paiYarn code
  2. Refactor paiK8S to PAI

@SparkSnail
Merge pull request #251 from microsoft/master
dcd2ffd 
Merge master
@SparkSnail
Merge pull request #252 from microsoft/master
3b8b6fb 
merge master
@SparkSnail
Merge pull request #253 from microsoft/master
916e444 
merge master
@SparkSnail
Merge pull request #254 from microsoft/master
caeffb8 
merge master
@SparkSnail
Merge pull request #255 from microsoft/master
57c300e 
merge master
@SparkSnail
Merge pull request #257 from microsoft/master
65660e6 
merge master
@SparkSnail
Merge pull request #258 from microsoft/master
9376d6a 
merge master
@SparkSnail
Merge pull request #259 from microsoft/master
5fef3cf 
Chinese translation (microsoft#2458)
@SparkSnail
Merge pull request #261 from microsoft/master
5544ae8 
merge master
@SparkSnail
Merge pull request #262 from microsoft/master
f9fdfee 
merge master
@SparkSnail
add trial job detail link
c5e26ef
@SparkSnail
Merge branch 'master' of https://github.com/SparkSnail/nni
10a04ba
@SparkSnail
Merge pull request #263 from microsoft/master
aa64fe6 
merge master
@SparkSnail
Merge branch 'master' of https://github.com/SparkSnail/nni
4ed907f
@SparkSnail
Merge pull request #264 from microsoft/master
c6a5f8c 
merge master
@SparkSnail
Merge pull request #265 from microsoft/master
68abe2f 
merge master
@SparkSnail
Merge branch 'master' of https://github.com/SparkSnail/nni
c2b50d2
@SparkSnail
Merge pull request #266 from microsoft/master
14e9619 
merge master
@SparkSnail
Merge pull request #267 from microsoft/master
f69e206 
merge master
@SparkSnail
Merge branch 'master' of https://github.com/SparkSnail/nni
a5bb753
@SparkSnail
Merge pull request #270 from microsoft/master
12ef0aa 
merge master
@SparkSnail
Merge branch 'master' of https://github.com/SparkSnail/nni
7600a0f
@SparkSnail
Merge pull request #271 from microsoft/master
ddcf229 
merge master
@SparkSnail
Merge branch 'master' of https://github.com/SparkSnail/nni
bd327d4
@SparkSnail
Merge pull request #272 from microsoft/master
c4f6e66 
merge master
@SparkSnail
Merge branch 'master' of https://github.com/SparkSnail/nni
da2d1c4
@SparkSnail
Merge pull request #273 from microsoft/master
88f8c1b 
merge master
@SparkSnail
Merge branch 'master' of https://github.com/SparkSnail/nni
b59d0e2
@SparkSnail
Merge pull request #274 from microsoft/master
7eb15f8 
Merge master
@SparkSnail
Merge pull request #275 from microsoft/master
f73367f 
merge master
liuzhe-lz
liuzhe-lz reviewed Jan 25, 2021
View reviewed changes
nni/tools/nnictl/config_schema.py
@@ -569,7 +556,7 @@ def validate_pai_config_path(self, experiment_config):

def validate_pai_trial_conifg(self, experiment_config):
'''validate the trial config in pai platform'''
if experiment_config.get('trainingServicePlatform') in ['pai', 'paiYarn']:
if experiment_config.get('trainingServicePlatform') in ['pai']:
Copy link
Contributor

Choose a reason for hiding this comment

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

Seems this if is no longer needed.

@J-shang J-shang mentioned this pull request Jan 25, 2021
Closed
94 tasks
@SparkSnail SparkSnail merged commit 6f27204 into microsoft:master Jan 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@J-shang J-shang J-shang approved these changes

@liuzhe-lz liuzhe-lz liuzhe-lz approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@SparkSnail @J-shang @liuzhe-lz