From 47704c54d7bc02bb8a568386564e7687ab775b39 Mon Sep 17 00:00:00 2001 From: SebastianMorawiec <108305907+SebastianMorawiec@users.noreply.github.com> Date: Tue, 21 May 2024 12:59:58 +0200 Subject: [PATCH] Feat!: try to minimize dependencies taken from ray (#410) # The problem Installing ray dashboard brings in a lot of dependencies that we might not need. # This PR's solution Limit dependencies we install by default. # Checklist _Check that this PR satisfies the following items:_ - [x] Tests have been added for new features/changed behavior (if no new features have been added, check the box). - [x] The [changelog file](CHANGELOG.md) has been updated with a user-readable description of the changes (if the change isn't visible to the user in any way, check the box). - [x] The PR's title is prefixed with `[!]:` - [x] The PR is linked to a JIRA ticket (if there's no suitable ticket, check the box). --------- Co-authored-by: Alexander Juda <6004040+alexjuda@users.noreply.github.com> --- projects/orquestra-sdk/pyproject.toml | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/projects/orquestra-sdk/pyproject.toml b/projects/orquestra-sdk/pyproject.toml index a0b00df31..0ebb50749 100644 --- a/projects/orquestra-sdk/pyproject.toml +++ b/projects/orquestra-sdk/pyproject.toml @@ -72,10 +72,18 @@ dependencies = [ [project.optional-dependencies] ray = [ - "ray[default]==2.9.0", + "ray==2.9.0", + "virtualenv>=20.0.24, !=20.21.1", # Ray depends on this, but it's missing from the base requirements. "async-timeout", # Ray depends on this, but its missing from ray requirements. Can be removed when https://github.com/ray-project/ray/issues/41267 is fixed. ] -all = ["orquestra-sdk[ray]"] + +dashboard = [ + "orquestra-sdk[ray]", + "ray[default]", +] + +all = ["orquestra-sdk[dashboard]"] + # Docs-specific deps. This list should match the requirements we're using in # the multi-repo docs builds. docs = [