-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
[Doc] Fix Title of the Transformers GLUE example #39605
Conversation
Signed-off-by: woshiyyya <[email protected]>
Signed-off-by: woshiyyya <[email protected]>
Signed-off-by: woshiyyya <[email protected]>
python/ray/train/examples/accelerate/accelerate_torch_trainer_no_raydata.py
Outdated
Show resolved
Hide resolved
@@ -19,6 +19,7 @@ | |||
set_seed, | |||
) | |||
|
|||
import ray.data |
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.
Does the example not work without this?
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.
Ray Data has a lazy import defined here, but I've never really understood if this should be something done by all libraries, no libraries, or some libraries.
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.
I've tried that after we import ray.train
, we can freely call any methods of ray.data
. Probably because we imported ray data in ray 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.
Ray Data can be accessed after import ray directly. Examples here
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.
oh that makes sense now. Ray train import ray, then ray.data is lazily imported.
Signed-off-by: woshiyyya <[email protected]>
Signed-off-by: woshiyyya <[email protected]>
Signed-off-by: woshiyyya <[email protected]>
* [Doc] Fix Title of the Transformers GLUE example (#39605) Signed-off-by: woshiyyya <[email protected]> * [docs][train]Make Train example titles, heading more consistent (#39606) Signed-off-by: angelinalg <[email protected]> Co-authored-by: Yunxuan Xiao <[email protected]> Co-authored-by: matthewdeng <[email protected]> * [docs][clusters] Change title of RayService doc to Deploy Ray Serve Apps (#39641) Signed-off-by: angelinalg <[email protected]> --------- Signed-off-by: woshiyyya <[email protected]> Signed-off-by: angelinalg <[email protected]> Co-authored-by: Yunxuan Xiao <[email protected]> Co-authored-by: Yunxuan Xiao <[email protected]> Co-authored-by: matthewdeng <[email protected]>
Signed-off-by: woshiyyya <[email protected]>
Signed-off-by: woshiyyya <[email protected]> Signed-off-by: Victor <[email protected]>
Why are these changes needed?
Fix the title of the GLUE example, also added missing
ray.data
imports.Related issue number
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.