Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

WIP feat(suggestion): Support One-shot NAS suggestion #9

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

WIP feat(suggestion): Support One-shot NAS suggestion #9

wants to merge 5 commits into from

Conversation

anchovYu
Copy link
Collaborator

No description provided.

anchovYu added 5 commits May 12, 2019 23:37
@anchovYu
Suggestion-0.1: Add suggestion controller
001cd34
@anchovYu
Suggestion-0.1: Add controller basic logic, w/o logger or recorder, w…
0b4070f 
…/o cluster test
@anchovYu
Suggestion-0.1: Add logger or recorder, w/o cluster test
49503b5
@anchovYu
Suggestion-0.1: Test in cluster, with CRD creation and deployment del…
0b8e3f1 
…etion
@anchovYu
DARTS suggestion path verification progress
75cd087 
1. implement exporting to ONNX of darts suggestion
2. implement importing ONNX to caffe2 in trial code (not included)
3. implement step1's GRPC
4. local test on GRPC getSuggestions

TODO:
5. build step1 into image, run in katib to test GRPC
6. build step2 into image, run trial in katib to test the whole path
7. re-design getSuggestions GRPC, re-implement Experiment and etc.
@caicloud-bot
Copy link

@anchovYu: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@caicloud-bot caicloud-bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. labels May 27, 2019
@caicloud-bot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: gaocegege

Assign the PR to them by writing /assign @gaocegege in a comment when ready.

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@anchovYu
Copy link
Collaborator Author

/release-note-none

@caicloud-bot caicloud-bot added caicloud-cla: yes Indicates the PR's author has not signed the Caicloud CLA. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. release-note-none Denotes a PR that doesn't merit a release note. and removed do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. labels May 27, 2019
@gaocegege gaocegege mentioned this pull request Apr 27, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
caicloud-cla: yes Indicates the PR's author has not signed the Caicloud CLA. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. release-note-none Denotes a PR that doesn't merit a release note. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@anchovYu @caicloud-bot