guan404ming commented on code in PR #1154:
URL: https://github.com/apache/mahout/pull/1154#discussion_r2939728217
##########
Makefile:
##########
@@ -39,8 +39,39 @@ ifeq ($(HAS_NVIDIA),yes)
else
@echo "[SKIP] No NVIDIA GPU detected, skipping maturin develop"
endif
- uv run pytest -n auto --cov=qumat --cov=qumat_qdp
--cov-report=term-missing --cov-report=html:htmlcov
+ uv run pytest --cov=qumat --cov=qumat_qdp --cov-report=term-missing
--cov-report=html:htmlcov
tests: test_rust test_python
pre-commit: setup-test-python
uv run pre-commit run --all-files
+
+setup-benchmark: setup-test-python
+ifeq ($(HAS_NVIDIA),yes)
+ @echo "[INFO] Setting up benchmark environment..."
+ uv sync --group dev --extra qdp
+ uv sync --project qdp/qdp-python --group benchmark --active
+ unset CONDA_PREFIX && uv run --active maturin develop --manifest-path
qdp/qdp-python/Cargo.toml
+else
+ @echo "[SKIP] No NVIDIA GPU detected, skipping maturin develop"
+ @echo "[INFO] Setting up benchmark environment (CPU-only)..."
+ uv sync --project qdp/qdp-python --group benchmark --active
+endif
+
+benchmark: setup-benchmark
Review Comment:
Should we rename to setup-benchmark or benchmark-info since it doesn't run
any benchmarks?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
