-
Notifications
You must be signed in to change notification settings - Fork 315
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
[Autotuner] Improve python package management #2859
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Jack Luar <[email protected]>
Signed-off-by: Jack Luar <[email protected]>
…ards Signed-off-by: Jack Luar <[email protected]>
requires-python = ">= 3.8" | ||
dynamic = ["dependencies", "optional-dependencies"] | ||
dependencies = [ | ||
"ray[default,tune] >= 2.10, <= 2.23", |
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.
To close #2859 we need at least 2.40
. We can continue with the PR, but this will not be enough to close the issue.
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.
Removed the issue link. I will fix some ray compatibility issues, marking this as a draft
…p/ray) Signed-off-by: Jack Luar <[email protected]>
…tmp instead of local) Signed-off-by: Jack Luar <[email protected]>
…ctly Signed-off-by: Jack Luar <[email protected]>
Signed-off-by: Jack Luar <[email protected]>
Signed-off-by: Jack Luar <[email protected]>
Signed-off-by: Jack Luar <[email protected]>
… reliably - currently under gp/dp constraint -> 0.625 of the configurations are valid. if we use 5 samples -> only 0.625*0.625*5 ~ 1.9 samples will be valid. - if we use 8 samples ~ 3.1 samples will be valid. Signed-off-by: Jack Luar <[email protected]>
uv
, a fast rust based package manager for python.Fixes #2888