-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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] migrate AIR API references #38632
Conversation
Signed-off-by: Matthew Deng <matt@anyscale.com>
Are we removing all the non-tabular preprocessor API references from the documentation? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks!
@@ -0,0 +1,3 @@ | |||
from ray.air.integrations.comet import CometLoggerCallback | |||
|
|||
CometLoggerCallback.__module__ = "ray.tune.logger.comet" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not totally sure what this does, so skipping this part of the review
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This just makes the path show up properly
>>> from ray.tune.logger.comet import CometLoggerCallback
>>> CometLoggerCallback
<class 'ray.tune.logger.comet.CometLoggerCallback'>
@@ -1,25 +0,0 @@ | |||
|
|||
Ray AIR Configurations | |||
====================== |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
assuming this already got moved to train?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep, the configs are in the Train/Tune API references already.
|
||
Experiment Tracking Integrations | ||
================================ | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
assuming this file moved to train?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The callbacks are in Tune, and we are choosing to not recommend to setup
methods
Signed-off-by: Matthew Deng <matt@anyscale.com>
Signed-off-by: Matthew Deng <matt@anyscale.com>
Signed-off-by: Matthew Deng <matt@anyscale.com> Signed-off-by: e428265 <arvind.chandramouli@lmco.com>
Signed-off-by: Matthew Deng <matt@anyscale.com> Signed-off-by: Victor <vctr.y.m@example.com>
Why are these changes needed?
This PR disbands the AIR API References page. Most API references were already defined within their individual libraries.
Additional changes include:
Preprocessors
to Data.LoggerCallback
s to Tune.Related issue number
#38365
Checks
git commit -s
) in this PR.scripts/format.sh
to lint the changes in this PR.method in Tune, I've added it in
doc/source/tune/api/
under thecorresponding
.rst
file.