[AIR] More modularity for Trainers #28650

Yard1 · 2022-09-20T19:12:15Z

Signed-off-by: Antoni Baum [email protected]

Why are these changes needed?

Internal refactor to make user/developer subclassing easier for some trainers.

Closes #28616

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 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: Antoni Baum <[email protected]>

Yard1 added 3 commits September 20, 2022 19:10

[AIR] More modularity for Trainers, use session.

83aac65

Signed-off-by: Antoni Baum <[email protected]>

Fix

7b06742

Signed-off-by: Antoni Baum <[email protected]>

Go back to tune.report

2a704b7

Signed-off-by: Antoni Baum <[email protected]>

Yard1 changed the title ~~[AIR] More modularity for Trainers, use session.~~ [AIR] More modularity for Trainers Sep 20, 2022

Yard1 requested review from krfricke and amogkam September 21, 2022 15:42

Yard1 assigned amogkam and krfricke Sep 21, 2022

Yard1 marked this pull request as ready for review September 21, 2022 15:42

krfricke approved these changes Sep 21, 2022

View reviewed changes

krfricke merged commit 1e54fff into ray-project:master Sep 21, 2022

Yard1 deleted the air_trainers_more_modularity branch September 21, 2022 16:35