Skip to content

Conversation

@dentiny
Copy link
Contributor

@dentiny dentiny commented Nov 7, 2024

This PR should be merged after all previous PRs, which documents all existing runtime env setup with uv.

@dentiny dentiny requested review from jjyao and rynewang November 7, 2024 22:22
@dentiny dentiny requested a review from a team as a code owner November 7, 2024 22:22
When specifying a path to a ``requirements.txt`` file, the file must be present on your local machine and it must be a valid absolute path or relative filepath relative to your local current working directory, *not* relative to the ``working_dir`` specified in the ``runtime_env``.
Furthermore, referencing local files *within* a ``requirements.txt`` file isn't directly supported (e.g., ``-r ./my-laptop/more-requirements.txt``, ``./my-pkg.whl``). Instead, use the ``${RAY_RUNTIME_ENV_CREATE_WORKING_DIR}`` environment variable in the creation process. For example, use ``-r ${RAY_RUNTIME_ENV_CREATE_WORKING_DIR}/my-laptop/more-requirements.txt`` or ``${RAY_RUNTIME_ENV_CREATE_WORKING_DIR}/my-pkg.whl`` to reference local files, while ensuring they're in the ``working_dir``.

- ``uv`` (dict | List[str] | str): Either (1) a list of uv `requirements specifiers <https://pip.pypa.io/en/stable/cli/pip_install/#requirement-specifiers>`_, (2) a string containing the path to a local uv
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Most of the parts are copy & paste from pip documentation.

@dentiny dentiny requested a review from pcmoritz November 8, 2024 03:22
Signed-off-by: dentiny <[email protected]>
@jjyao jjyao added the go add ONLY when ready to merge, run all tests label Nov 8, 2024
@jjyao jjyao merged commit 0251a3e into ray-project:master Nov 12, 2024
6 checks passed
JP-sDEV pushed a commit to JP-sDEV/ray that referenced this pull request Nov 14, 2024
mohitjain2504 pushed a commit to mohitjain2504/ray that referenced this pull request Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-backlog go add ONLY when ready to merge, run all tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants