python / python/pyperformance

Only Try a Unique Venv If There Is a Conflict

Open
#157 1 comment 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Python
Stars
1k
Forks
203
Avg merge
1h 20m
Merged PRs (30d)
2

Description

Currently any failure when installing a benchmark's requirements triggers the creation of a unique venv for that benchmark and a retry of the install. The only failure for which we need to create a unique venv is when there is a conflict with the version of already installed packages. Adjusting the code along those lines would speed things up when their are failures (e.g. wheel cannot build) and produce less noise.

Possible solutions:

  • parse the text of the first install attempt to see if it a conflict
    • this is fragile, as well as requiring that we "tee" the output of the "pip install" command
  • instead of trying the common venv first, track the installed versions and use the common venv only if the benchmark's requirements are compatible
    • #158 identifies several approaches to tracking installed versions
    • this means we need to add some logic for checking requirements compatibility, though there is probably a good enough library out there (maybe in the stdlib)

Also see https://github.com/python/pyperformance/issues/144#issuecomment-1075822776.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Trace the benchmark requirements-installation flow and review issue #158 for approaches to tracking installed versions. The change is complete when a unique virtual environment is used only for package-version conflicts, while other installation failures are reported without an automatic retry.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
performance, tooling
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.