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.

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

[docs] Cleanup for other AIR concepts #39400

Merged
merged 8 commits into from
Sep 8, 2023

Conversation

richardliaw
Copy link
Contributor

@richardliaw richardliaw commented Sep 7, 2023

Why are these changes needed?

Removes additional references to AIR in diagrams and text.

Related issue number

Checks

  • I've signed off every commit(by using the -s flag, i.e., git commit -s) in this PR.
  • I've run scripts/format.sh to lint the changes in this PR.
  • I've included any doc changes needed for https://docs.ray.io/en/master/.
    • I've added any new APIs to the API Reference. For example, if I added a
      method in Tune, I've added it in doc/source/tune/api/ under the
      corresponding .rst file.
  • I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failures at https://flakey-tests.ray.io/
  • Testing Strategy
    • Unit tests
    • Release tests
    • This PR is not tested :(

Copy link
Contributor

@architkulkarni architkulkarni left a comment

Choose a reason for hiding this comment

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

/cluster changes looks good, approving as codeowner

Copy link
Contributor

@ericl ericl left a comment

Choose a reason for hiding this comment

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

stamp

Copy link
Contributor

@matthewdeng matthewdeng left a comment

Choose a reason for hiding this comment

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

@richardliaw richardliaw merged commit 7702453 into ray-project:master Sep 8, 2023
12 of 20 checks passed
@angelinalg angelinalg added docs An issue or change related to documentation v2.7.0-pick labels Sep 8, 2023
angelinalg pushed a commit to angelinalg/ray that referenced this pull request Sep 9, 2023
GeneDer pushed a commit that referenced this pull request Sep 9, 2023
#39510)

* Update metrics.md (#38512)

1. there are 3 dashboards in the folder now. Refer to the folder instead of only 1 dashboard
2. include "Copy" since people need to copy this from the head node to the Grafana server

Signed-off-by: Huaiwei Sun <[email protected]>

* polish observability (o11y) docs (#39069)

Signed-off-by: Huaiwei Sun <[email protected]>
Co-authored-by: angelinalg <[email protected]>
Co-authored-by: matthewdeng <[email protected]>

* [Doc] Unbold "Use Cases" in sidebar (#39295)

Signed-off-by: pdmurray <[email protected]>

* [docs] Cleanup for other AIR concepts (#39400)

* [doc][clusters] add doc for setting up Ray and K8s (#39408)

---------

Signed-off-by: Huaiwei Sun <[email protected]>
Signed-off-by: pdmurray <[email protected]>
Co-authored-by: Huaiwei Sun <[email protected]>
Co-authored-by: matthewdeng <[email protected]>
Co-authored-by: Peyton Murray <[email protected]>
Co-authored-by: Richard Liaw <[email protected]>
jimthompson5802 pushed a commit to jimthompson5802/ray that referenced this pull request Sep 12, 2023
vymao pushed a commit to vymao/ray that referenced this pull request Oct 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs An issue or change related to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants