search

kubeflow / blog

Kubeflow blog based on fastpages
https://blog.kubeflow.org
Apache License 2.0
17 stars 40 forks source link

Katib 0.10 Release Blog Post #71

Closed andreyvelich closed 3 years ago

andreyvelich commented 3 years ago

I added Blog Post: "Kubeflow Katib: Scalable, Portable and Cloud Native System for AutoML". Name is under discussion, if you have any other options please let me know.

/cc @jbottum @gaocegege @johnugeorge @castrojo @amsaha

k8s-ci-robot commented 3 years ago

@andreyvelich: GitHub didn't allow me to request PR reviews from the following users: castrojo.

Note that only kubeflow members and repo collaborators can review this PR, and authors cannot review their own PRs.

In response to [this](https://github.com/kubeflow/blog/pull/71): >I added Blog Post: "Kubeflow Katib: Scalable, Portable and Cloud Native System for AutoML". >Name is under discussion, if you have any other options please let me know. > >/cc @jbottum @gaocegege @johnugeorge @castrojo @amsaha Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
k8s-ci-robot commented 3 years ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: andreyvelich To complete the pull request process, please assign theadactyl after the PR has been reviewed. You can assign the PR to them by writing /assign @theadactyl in a comment when ready.

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

Needs approval from an approver in each of these files: - **[OWNERS](https://github.com/kubeflow/blog/blob/master/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
andreyvelich commented 3 years ago

@kramachandran @theadactyl Can you help with this Blog Post review, please ?

theadactyl commented 3 years ago

Hi! Thanks for tagging me. Can we get an LGTM from a subject matter expert aside from blog author? I think there are some copy edits needed here as well, which I'd love help with. Otherwise looks good to me -- happy to approve when those are taken care of.

andreyvelich commented 3 years ago

Hi! Thanks for tagging me. Can we get an LGTM from a subject matter expert aside from blog author? I think there are some copy edits needed here as well, which I'd love help with. Otherwise looks good to me -- happy to approve when those are taken care of.

Thank you @theadactyl! Please let me know which parts can be modified in this Blog from your point of view.

andreyvelich commented 3 years ago

I removed WIP status. /hold for the review.

andreyvelich commented 3 years ago

Thank you so much for the review @theadactyl! I addressed your comments.

jbottum commented 3 years ago

@andreyvelich very nice effort. I have 3 comments.

(this sentence might need some commas or broken in to two sentences) - Since Katib can execute various Kubernetes resources, users are able to run Experiments which can optimize not only ML models but also any software, code or program to make it more efficient with optimization algorithms provided by Katib.

(should "perform" be "provide a"?) We are continually working on the new Katib UI to perform better User Experience and native integration with Kubeflow central dashboard.

(this sentence probably needs to be broken into two or more sentences...and I struggle with best and worse in the same sentence) : Currently, Katib supports the Median Stopping Rule which stops a pending Trial X at step S if the Trial's best objective value by step S is worse than the median value of the running averages of all completed Trials' objectives reported up to step S. Suggestion.... Currently, Katib supports the Median Stopping Rule. The Medium Stopping rule stops if the Trial's best objective is lower than the median value of all Trials' objectives completed to that point.

andreyvelich commented 3 years ago

Thank you for the comments @jbottum! What do you think ?

(this sentence might need some commas or broken in to two sentences) - Since Katib can execute various Kubernetes resources, users are able to run Experiments which can optimize not only ML models but also any software, code or program to make it more efficient with optimization algorithms provided by Katib.

Since Katib can execute various Kubernetes resources, users are able to run not only ML models optimization Experiments. They can also enhance any software, code or program to make it more efficient with optimization algorithms provided by Katib.

(should "perform" be "provide a"?) We are continually working on the new Katib UI to perform better User Experience and native integration with Kubeflow central dashboard.

Make sense, changed.

(this sentence probably needs to be broken into two or more sentences...and I struggle with best and worse in the same sentence) : Currently, Katib supports the Median Stopping Rule which stops a pending Trial X at step S if the Trial's best objective value by step S is worse than the median value of the running averages of all completed Trials' objectives reported up to step S. Suggestion.... Currently, Katib supports the Median Stopping Rule. The Medium Stopping rule stops if the Trial's best objective is lower than the median value of all Trials' objectives completed to that point.

Thanks! What do you think about this "Currently, Katib supports the Median Stopping Rule. The Medium Stopping rule stops a running Trial at the step S if the Trial's best objective value is lower than the median value of all succeeded Trials' objectives reported up to that step S."

google-oss-robot commented 3 years ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: andreyvelich, theadactyl

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: - ~~[OWNERS](https://github.com/kubeflow/blog/blob/master/OWNERS)~~ [theadactyl] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
theadactyl commented 3 years ago

LMK when all stakeholders are ready for merge.

andreyvelich commented 3 years ago

/assign @gaocegege @johnugeorge @jbottum

jbottum commented 3 years ago

/lgtm

andreyvelich commented 3 years ago

@gaocegege @johnugeorge If you are fine with the changes, please can you give your lgtm and we can move forward?

johnugeorge commented 3 years ago

/lgtm

andreyvelich commented 3 years ago

Thanks everyone for the help on this blog! /hold cancel

theadactyl commented 3 years ago

Congrats!

On Wed, Mar 10, 2021 at 2:38 AM Google OSS Prow Robot < notifications@github.com> wrote:

Merged #71 https://github.com/kubeflow/blog/pull/71 into master.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/kubeflow/blog/pull/71#event-4435280602, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABREJVM3EDF64ZRJCTKFGMTTC442DANCNFSM4YNYNQWA .