-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update asv version. #86
base: main
Are you sure you want to change the base?
Conversation
Still seeing this error:
Trying to see if I can fix it... |
This reverts commit 9b2ce08.
…. Also added the required libmambapy dependency. https://asv.readthedocs.io/en/stable/asv.conf.json.html#conda-environment-file
This reverts commit bd41fc8.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I propose to use Python 3.12 and proposed small, non-blocking suggestions.
uses: actions/checkout@v3 | ||
- uses: actions/setup-python@v4 | ||
uses: actions/checkout@v4 | ||
- uses: actions/setup-python@v5 | ||
with: | ||
python-version: '3.11' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
python-version: '3.11' | |
python-version: '3.12' |
It seems ASV support Python 3.12 since version 0.6.2.
activate-environment: asv-bench | ||
environment-file: environment.yml | ||
- name: Install Requirements | ||
run: pip install -r requirements.txt |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
run: pip install -r requirements.txt | |
run: python -m pip install --requirement requirements.txt |
Using pip
as module is a best practice, and also long form for attributes.
asv machine --machine ubuntu-latest --yes | ||
echo '```' >> $GITHUB_STEP_SUMMARY | ||
asv continuous --interleave-processes -a processes=2 --split --show-stderr 'HEAD^' 'HEAD' |\ | ||
asv continuous -E conda:asv-bench --interleave-processes -a processes=2 --split --show-stderr 'HEAD^' 'HEAD' |\ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
asv continuous -E conda:asv-bench --interleave-processes -a processes=2 --split --show-stderr 'HEAD^' 'HEAD' |\ | |
asv continuous --environment conda:asv-bench --interleave-processes --attribute processes=2 --split --show-stderr 'HEAD^' 'HEAD' |\ |
Used long form attributes.
uses: actions/checkout@v3 | ||
- uses: actions/setup-python@v4 | ||
uses: actions/checkout@v4 | ||
- uses: actions/setup-python@v5 | ||
with: | ||
python-version: '3.11' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
python-version: '3.11' | |
python-version: '3.12' |
See above.
uses: actions/checkout@v3 | ||
- uses: actions/setup-python@v4 | ||
uses: actions/checkout@v4 | ||
- uses: actions/setup-python@v5 | ||
with: | ||
python-version: '3.11' | ||
- name: Install Requirements |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- name: Install Requirements | |
- name: Install Requirements | |
run: python -m pip install --requirement requirements.txt |
See above.
I wasn't able to select the line below in the suggestions, but you can delete it.
- pip | ||
- wheel | ||
- conda-build | ||
- libmambapy |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- pip | |
- wheel | |
- conda-build | |
- libmambapy | |
- conda-build | |
- libmambapy | |
- pip | |
- wheel |
I only sorted dependencies, it can help in managing them.
ASV 0.6.4 is released. https://pypi.org/project/asv/0.6.4/