Skip to content

Commit

Permalink
Feat!: try to minimize dependencies taken from ray (#410)
Browse files Browse the repository at this point in the history
# 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
`<feat/fix/chore/imp[rovement]/int[ernal]/docs>[!]:`
- [x] The PR is linked to a JIRA ticket (if there's no suitable ticket,
check the box).

---------

Co-authored-by: Alexander Juda <[email protected]>
  • Loading branch information
SebastianMorawiec and alexjuda authored May 21, 2024
1 parent 398ee23 commit 47704c5
Showing 1 changed file with 10 additions and 2 deletions.
12 changes: 10 additions & 2 deletions projects/orquestra-sdk/pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -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 = [
Expand Down

0 comments on commit 47704c5

Please sign in to comment.