Skip to content

[tune] remove TuneRichReporter#40169

Merged
matthewdeng merged 3 commits intoray-project:masterfrom
matthewdeng:tune-rich
Oct 17, 2023
Merged

[tune] remove TuneRichReporter#40169
matthewdeng merged 3 commits intoray-project:masterfrom
matthewdeng:tune-rich

Conversation

@matthewdeng
Copy link
Contributor

@matthewdeng matthewdeng commented Oct 6, 2023

Why are these changes needed?

Removing this since it's not being used/maintained. 😢

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

Signed-off-by: Matthew Deng <matt@anyscale.com>
matthewdeng and others added 2 commits October 17, 2023 10:12
Signed-off-by: matthewdeng <matt@anyscale.com>
@matthewdeng matthewdeng added the tests-ok The tagger certifies test failures are unrelated and assumes personal liability. label Oct 17, 2023
@matthewdeng matthewdeng merged commit c91ee0f into ray-project:master Oct 17, 2023
@matthewdeng matthewdeng deleted the tune-rich branch October 17, 2023 20:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests-ok The tagger certifies test failures are unrelated and assumes personal liability.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants