Skip to content

Commit 58fa932

Browse files
elliot-barnykdojo
authored andcommitted
[ci] sorting requirements and constraints for raydepsets (ray-project#58414)
Sorting requirements and constraints for raydepsets --------- Signed-off-by: elliot-barn <elliot.barnwell@anyscale.com> Signed-off-by: YK <1811651+ykdojo@users.noreply.github.com>
1 parent 8f27280 commit 58fa932

20 files changed

+80
-20
lines changed

ci/raydepsets/cli.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -318,10 +318,10 @@ def compile(
318318
if append_flags:
319319
args.extend(_flatten_flags(append_flags))
320320
if constraints:
321-
for constraint in constraints:
321+
for constraint in sorted(constraints):
322322
args.extend(["-c", constraint])
323323
if requirements:
324-
for requirement in requirements:
324+
for requirement in sorted(requirements):
325325
args.extend([requirement])
326326
if packages:
327327
# need to add a dash to process stdin

ci/raydepsets/tests/test_cli.py

Lines changed: 60 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -712,6 +712,66 @@ def test_compile_with_packages_and_requirements(self):
712712
output_text_valid = output_file_valid.read_text()
713713
assert output_text == output_text_valid
714714

715+
@patch("sys.stdout", new_callable=io.StringIO)
716+
def test_requirements_ordering(self, mock_stdout):
717+
with tempfile.TemporaryDirectory() as tmpdir:
718+
copy_data_to_tmpdir(tmpdir)
719+
save_packages_to_file(
720+
Path(tmpdir) / "requirements_expanded.txt",
721+
["six"],
722+
)
723+
save_packages_to_file(
724+
Path(tmpdir) / "requirements_compiled_test_expand.txt",
725+
["zipp"],
726+
)
727+
manager = _create_test_manager(tmpdir)
728+
manager.compile(
729+
constraints=["requirement_constraints_test.txt"],
730+
requirements=[
731+
"requirements_test.txt",
732+
"requirements_expanded.txt",
733+
"requirements_compiled_test_expand.txt",
734+
],
735+
append_flags=["--no-annotate", "--no-header"],
736+
name="requirements_ordering_test_depset",
737+
output="requirements_compiled_requirements_ordering.txt",
738+
)
739+
stdout = mock_stdout.getvalue()
740+
assert (
741+
"requirements_compiled_test_expand.txt requirements_expanded.txt requirements_test.txt"
742+
in stdout
743+
)
744+
745+
@patch("sys.stdout", new_callable=io.StringIO)
746+
def test_constraints_ordering(self, mock_stdout):
747+
with tempfile.TemporaryDirectory() as tmpdir:
748+
copy_data_to_tmpdir(tmpdir)
749+
save_packages_to_file(
750+
Path(tmpdir) / "requirements_expanded.txt",
751+
["six==1.17.0"],
752+
)
753+
save_packages_to_file(
754+
Path(tmpdir) / "requirements_compiled_test_expand.txt",
755+
["zipp==3.19.2"],
756+
)
757+
manager = _create_test_manager(tmpdir)
758+
manager.compile(
759+
requirements=["requirements_test.txt"],
760+
constraints=[
761+
"requirement_constraints_test.txt",
762+
"requirements_expanded.txt",
763+
"requirements_compiled_test_expand.txt",
764+
],
765+
append_flags=["--no-annotate", "--no-header"],
766+
name="constraints_ordering_test_depset",
767+
output="requirements_compiled_constraints_ordering.txt",
768+
)
769+
stdout = mock_stdout.getvalue()
770+
assert (
771+
"-c requirement_constraints_test.txt -c requirements_compiled_test_expand.txt -c requirements_expanded.txt"
772+
in stdout
773+
)
774+
715775
@patch("sys.stdout", new_callable=io.StringIO)
716776
def test_execute_pre_hook(self, mock_stdout):
717777
with tempfile.TemporaryDirectory() as tmpdir:

python/deplocks/llm/ray_test_py311_cpu.lock

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# This file was autogenerated by uv via the following command:
2-
# uv pip compile --generate-hashes --strip-extras --unsafe-package setuptools --index-url https://pypi.org/simple --index-strategy unsafe-best-match --no-strip-markers --emit-index-url --emit-find-links --python-version=3.11 --unsafe-package ray --python-platform=linux --extra-index-url https://download.pytorch.org/whl/cpu -c /tmp/ray-deps/requirements_compiled.txt python/requirements.txt python/requirements/cloud-requirements.txt python/requirements/base-test-requirements.txt -o python/deplocks/llm/ray_test_py311_cpu.lock
2+
# uv pip compile --generate-hashes --strip-extras --unsafe-package setuptools --index-url https://pypi.org/simple --index-strategy unsafe-best-match --no-strip-markers --emit-index-url --emit-find-links --python-version=3.11 --unsafe-package ray --python-platform=linux --extra-index-url https://download.pytorch.org/whl/cpu -c /tmp/ray-deps/requirements_compiled.txt python/requirements.txt python/requirements/base-test-requirements.txt python/requirements/cloud-requirements.txt -o python/deplocks/llm/ray_test_py311_cpu.lock
33
--index-url https://pypi.org/simple
44
--extra-index-url https://download.pytorch.org/whl/cpu
55

python/deplocks/llm/ray_test_py311_cu128.lock

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# This file was autogenerated by uv via the following command:
2-
# uv pip compile --generate-hashes --strip-extras --unsafe-package setuptools --index-url https://pypi.org/simple --index-strategy unsafe-best-match --no-strip-markers --emit-index-url --emit-find-links --python-version=3.11 --unsafe-package ray --python-platform=linux --extra-index-url https://download.pytorch.org/whl/cu128 -c /tmp/ray-deps/requirements_compiled.txt python/requirements.txt python/requirements/cloud-requirements.txt python/requirements/base-test-requirements.txt -o python/deplocks/llm/ray_test_py311_cu128.lock
2+
# uv pip compile --generate-hashes --strip-extras --unsafe-package setuptools --index-url https://pypi.org/simple --index-strategy unsafe-best-match --no-strip-markers --emit-index-url --emit-find-links --python-version=3.11 --unsafe-package ray --python-platform=linux --extra-index-url https://download.pytorch.org/whl/cu128 -c /tmp/ray-deps/requirements_compiled.txt python/requirements.txt python/requirements/base-test-requirements.txt python/requirements/cloud-requirements.txt -o python/deplocks/llm/ray_test_py311_cu128.lock
33
--index-url https://pypi.org/simple
44
--extra-index-url https://download.pytorch.org/whl/cu128
55

python/deplocks/llm/rayllm_test_py311_cpu.lock

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# This file was autogenerated by uv via the following command:
2-
# uv pip compile --generate-hashes --strip-extras --unsafe-package setuptools --index-url https://pypi.org/simple --index-strategy unsafe-best-match --no-strip-markers --emit-index-url --emit-find-links --python-version=3.11 --unsafe-package ray --python-platform=linux --extra-index-url https://download.pytorch.org/whl/cpu -c python/deplocks/llm/ray_test_py311_cpu.lock python/requirements.txt python/requirements/cloud-requirements.txt python/requirements/base-test-requirements.txt python/requirements/llm/llm-requirements.txt python/requirements/llm/llm-test-requirements.txt -o python/deplocks/llm/rayllm_test_py311_cpu.lock
2+
# uv pip compile --generate-hashes --strip-extras --unsafe-package setuptools --index-url https://pypi.org/simple --index-strategy unsafe-best-match --no-strip-markers --emit-index-url --emit-find-links --python-version=3.11 --unsafe-package ray --python-platform=linux --extra-index-url https://download.pytorch.org/whl/cpu -c python/deplocks/llm/ray_test_py311_cpu.lock python/requirements.txt python/requirements/base-test-requirements.txt python/requirements/cloud-requirements.txt python/requirements/llm/llm-requirements.txt python/requirements/llm/llm-test-requirements.txt -o python/deplocks/llm/rayllm_test_py311_cpu.lock
33
--index-url https://pypi.org/simple
44
--extra-index-url https://download.pytorch.org/whl/cpu
55

python/deplocks/llm/rayllm_test_py311_cu128.lock

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# This file was autogenerated by uv via the following command:
2-
# uv pip compile --generate-hashes --strip-extras --unsafe-package setuptools --index-url https://pypi.org/simple --index-strategy unsafe-best-match --no-strip-markers --emit-index-url --emit-find-links --python-version=3.11 --unsafe-package ray --python-platform=linux --extra-index-url https://download.pytorch.org/whl/cu128 -c python/deplocks/llm/ray_test_py311_cu128.lock python/requirements.txt python/requirements/cloud-requirements.txt python/requirements/base-test-requirements.txt python/requirements/llm/llm-requirements.txt python/requirements/llm/llm-test-requirements.txt -o python/deplocks/llm/rayllm_test_py311_cu128.lock
2+
# uv pip compile --generate-hashes --strip-extras --unsafe-package setuptools --index-url https://pypi.org/simple --index-strategy unsafe-best-match --no-strip-markers --emit-index-url --emit-find-links --python-version=3.11 --unsafe-package ray --python-platform=linux --extra-index-url https://download.pytorch.org/whl/cu128 -c python/deplocks/llm/ray_test_py311_cu128.lock python/requirements.txt python/requirements/base-test-requirements.txt python/requirements/cloud-requirements.txt python/requirements/llm/llm-requirements.txt python/requirements/llm/llm-test-requirements.txt -o python/deplocks/llm/rayllm_test_py311_cu128.lock
33
--index-url https://pypi.org/simple
44
--extra-index-url https://download.pytorch.org/whl/cu128
55

release/ray_release/byod/audio_transcription_py3.10.lock

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# This file was autogenerated by uv via the following command:
2-
# uv pip compile --generate-hashes --strip-extras --unsafe-package setuptools --index-url https://pypi.org/simple --index-strategy unsafe-best-match --no-strip-markers --emit-index-url --emit-find-links --python-version=3.10 --unsafe-package ray --python-platform=linux release/ray_release/byod/ray_dev_py3.10.in docker/base-deps/requirements.in docker/base-extra/requirements.in release/nightly_tests/multimodal_inference_benchmarks/audio_transcription/requirements.in -o release/ray_release/byod/audio_transcription_py3.10.lock
2+
# uv pip compile --generate-hashes --strip-extras --unsafe-package setuptools --index-url https://pypi.org/simple --index-strategy unsafe-best-match --no-strip-markers --emit-index-url --emit-find-links --python-version=3.10 --unsafe-package ray --python-platform=linux docker/base-deps/requirements.in docker/base-extra/requirements.in release/nightly_tests/multimodal_inference_benchmarks/audio_transcription/requirements.in release/ray_release/byod/ray_dev_py3.10.in -o release/ray_release/byod/audio_transcription_py3.10.lock
33
--index-url https://pypi.org/simple
44

55
absl-py==1.4.0 \

release/ray_release/byod/audio_transcription_py3.9.lock

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# This file was autogenerated by uv via the following command:
2-
# uv pip compile --generate-hashes --strip-extras --unsafe-package setuptools --index-url https://pypi.org/simple --index-strategy unsafe-best-match --no-strip-markers --emit-index-url --emit-find-links --python-version=3.9 --unsafe-package ray --python-platform=linux release/ray_release/byod/ray_dev_py3.9.in docker/base-deps/requirements.in docker/base-extra/requirements.in release/nightly_tests/multimodal_inference_benchmarks/audio_transcription/requirements.in -o release/ray_release/byod/audio_transcription_py3.9.lock
2+
# uv pip compile --generate-hashes --strip-extras --unsafe-package setuptools --index-url https://pypi.org/simple --index-strategy unsafe-best-match --no-strip-markers --emit-index-url --emit-find-links --python-version=3.9 --unsafe-package ray --python-platform=linux docker/base-deps/requirements.in docker/base-extra/requirements.in release/nightly_tests/multimodal_inference_benchmarks/audio_transcription/requirements.in release/ray_release/byod/ray_dev_py3.9.in -o release/ray_release/byod/audio_transcription_py3.9.lock
33
--index-url https://pypi.org/simple
44

55
accelerate==1.10.1 \

release/ray_release/byod/ray_base_extra_testdeps_cuda_py3.10.lock

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# This file was autogenerated by uv via the following command:
2-
# uv pip compile --generate-hashes --strip-extras --unsafe-package setuptools --index-url https://pypi.org/simple --index-strategy unsafe-best-match --no-strip-markers --emit-index-url --emit-find-links --extra-index-url https://download.pytorch.org/whl/cu128 --unsafe-package ray --python-version=3.10 --python-platform=linux -c /tmp/ray-deps/requirements_compiled.txt release/ray_release/byod/ray_dev_py3.10.in release/ray_release/byod/requirements_byod_3.10.in docker/base-deps/requirements.in docker/base-extra/requirements.in -o release/ray_release/byod/ray_base_extra_testdeps_cuda_py3.10.lock
2+
# uv pip compile --generate-hashes --strip-extras --unsafe-package setuptools --index-url https://pypi.org/simple --index-strategy unsafe-best-match --no-strip-markers --emit-index-url --emit-find-links --extra-index-url https://download.pytorch.org/whl/cu128 --unsafe-package ray --python-version=3.10 --python-platform=linux -c /tmp/ray-deps/requirements_compiled.txt docker/base-deps/requirements.in docker/base-extra/requirements.in release/ray_release/byod/ray_dev_py3.10.in release/ray_release/byod/requirements_byod_3.10.in -o release/ray_release/byod/ray_base_extra_testdeps_cuda_py3.10.lock
33
--index-url https://pypi.org/simple
44
--extra-index-url https://download.pytorch.org/whl/cu128
55

release/ray_release/byod/ray_base_extra_testdeps_cuda_py3.11.lock

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# This file was autogenerated by uv via the following command:
2-
# uv pip compile --generate-hashes --strip-extras --unsafe-package setuptools --index-url https://pypi.org/simple --index-strategy unsafe-best-match --no-strip-markers --emit-index-url --emit-find-links --extra-index-url https://download.pytorch.org/whl/cu128 --unsafe-package ray --python-version=3.11 --python-platform=linux -c /tmp/ray-deps/requirements_compiled.txt release/ray_release/byod/ray_dev_py3.11.in release/ray_release/byod/requirements_byod_3.11.in docker/base-deps/requirements.in docker/base-extra/requirements.in -o release/ray_release/byod/ray_base_extra_testdeps_cuda_py3.11.lock
2+
# uv pip compile --generate-hashes --strip-extras --unsafe-package setuptools --index-url https://pypi.org/simple --index-strategy unsafe-best-match --no-strip-markers --emit-index-url --emit-find-links --extra-index-url https://download.pytorch.org/whl/cu128 --unsafe-package ray --python-version=3.11 --python-platform=linux -c /tmp/ray-deps/requirements_compiled.txt docker/base-deps/requirements.in docker/base-extra/requirements.in release/ray_release/byod/ray_dev_py3.11.in release/ray_release/byod/requirements_byod_3.11.in -o release/ray_release/byod/ray_base_extra_testdeps_cuda_py3.11.lock
33
--index-url https://pypi.org/simple
44
--extra-index-url https://download.pytorch.org/whl/cu128
55

0 commit comments

Comments
 (0)