-
Notifications
You must be signed in to change notification settings - Fork 651
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-#6819: Update Modin on cluster documentation #6678
Conversation
e3bf55d
to
6bfbdb2
Compare
Signed-off-by: Anatoly Myachev <anatoly.myachev@intel.com>
6bfbdb2
to
1ca9a8b
Compare
.. _`Ray's autoscaler options`: https://docs.ray.io/en/latest/cluster/vms/references/ray-cluster-configuration.html#cluster-config | ||
.. _`Ray's cluster docs`: https://docs.ray.io/en/latest/cluster/getting-started.html | ||
.. _`NYC Taxi dataset`: https://modin-datasets.s3.amazonaws.com/testing/yellow_tripdata_2015-01.csv | ||
.. _`Modin's cluster setup config`: https://github.com/modin-project/modin/blob/master/examples/tutorial/jupyter/execution/pandas_on_ray/cluster/modin-cluster.yaml |
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.
It seems this file is stale and we should update it as part of a separate issue. Actually, we would to need update cluster modes for all distr backends - Ray, Dask, MPI.
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.
It seems this file is stale and we should update it as part of a separate issue.
Indeed. Maybe @Retribution98 can look at it? Here one will need to create a cluster in AWS.
Actually, we would to need update cluster modes for all distr backends - Ray, Dask, MPI.
Don't get it.
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.
It seems this file is stale and we should update it as part of a separate issue.
Indeed. Maybe @Retribution98 can look at it? Here one will need to create a cluster in AWS.
We will schedule this offline.
Actually, we would to need update cluster modes for all distr backends - Ray, Dask, MPI.
Don't get it.
I mean docs/examples regarding Modin on cluster.
# TODO: Minimum version of Ray - 1.13 | ||
# `if` branch can be deleted |
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.
Good catch. We can also remove IsExperimental import in this file.
https://github.com/modin-project/modin/blob/324099d8737ead092e4acacfe03b1caa82e01986/modin/utils.py#L52C34-L52C48
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 wonder why flake8
does not see that this import is not used. Hm
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.
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
Co-authored-by: Iaroslav Igoshev <Poolliver868@mail.ru>
What do these changes do?
flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
git commit -s
docs/development/architecture.rst
is up-to-date