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

[tune/docs] Remove more reference to AIR concepts #39569

Merged
merged 5 commits into from
Sep 12, 2023
Merged

[tune/docs] Remove more reference to AIR concepts #39569

merged 5 commits into from
Sep 12, 2023

Conversation

krfricke
Copy link
Contributor

@krfricke krfricke commented Sep 12, 2023 •
edited
Loading

Why are these changes needed?

This PR removes remnants of AIR concepts in the (public-facing) Ray Tune documentation.

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 :(

[tune/docs] Remove more reference to AIR concepts
12d6e21 
Signed-off-by: Kai Fricke <kai@anyscale.com>
@krfricke krfricke marked this pull request as ready for review September 12, 2023 15:54
Kai Fricke added 2 commits September 12, 2023 11:33
Merge branch 'master' into doc/tune/air
09d70e5
train.report
a1e42f6 
Signed-off-by: Kai Fricke <kai@anyscale.com>
matthewdeng
matthewdeng reviewed Sep 12, 2023
View reviewed changes
Kai Fricke added 2 commits September 12, 2023 13:49
review
a820d3d 
Signed-off-by: Kai Fricke <kai@anyscale.com>
current module
d340c7b 
Signed-off-by: Kai Fricke <kai@anyscale.com>
@krfricke krfricke added the tests-ok The tagger certifies test failures are unrelated and assumes personal liability. label Sep 12, 2023
@krfricke krfricke merged commit b86f257 into ray-project:master Sep 12, 2023
@krfricke krfricke deleted the doc/tune/air branch September 12, 2023 23:16
vymao pushed a commit to vymao/ray that referenced this pull request Oct 11, 2023
@krfricke
[tune/docs] Remove more reference to AIR concepts (ray-project#39569)
16616d7 
This PR removes remnants of AIR concepts in the (public-facing) Ray Tune documentation.

Signed-off-by: Kai Fricke <kai@anyscale.com>
Signed-off-by: Victor <vctr.y.m@example.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@matthewdeng matthewdeng matthewdeng approved these changes

@richardliaw richardliaw Awaiting requested review from richardliaw

@xwjiang2010 xwjiang2010 Awaiting requested review from xwjiang2010

@amogkam amogkam Awaiting requested review from amogkam

@Yard1 Yard1 Awaiting requested review from Yard1

@maxpumperla maxpumperla Awaiting requested review from maxpumperla

Assignees

@matthewdeng matthewdeng

Labels
tests-ok The tagger certifies test failures are unrelated and assumes personal liability.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@krfricke @matthewdeng