Skip to content

Commit

Permalink
Fix asv installation command (#11758)
Browse files Browse the repository at this point in the history
The nightly benchmarking runs haven't been running recently due to
failures to import Qiskit.  This happened immediately after the import
poisoning.  The problem appears to be because the `all` extra in the
metadata references the `qiskit` package by name, since this is how we
avoid repeating ourselves in the definition of the extra.  However, as
we note in `pyproject.toml`, when called on a local wheel file, this
causes the resoution to install `qiskit` from PyPI, which means
overwriting the package with a different version.  We don't actually
need everything installed for the benchmarks, so we can just set the
rule to install only what is needed.
  • Loading branch information
jakelishman authored Feb 8, 2024
1 parent 3892871 commit 98aac28
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion asv.conf.json
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
"project_url": "https://qiskit.org",
"repo": ".",
"install_command": [
"in-dir={env_dir} python -m pip install {wheel_file}[all] python-constraint"
"in-dir={env_dir} python -m pip install {wheel_file}[csp-layout-pass]"
],
"uninstall_command": [
"return-code=any python -m pip uninstall -y qiskit qiskit-terra"
Expand Down

0 comments on commit 98aac28

Please sign in to comment.