Skip to content

fix: clean the poetry cache #4

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

Merged
merged 1 commit into from
Jul 20, 2022
Merged

fix: clean the poetry cache #4

merged 1 commit into from
Jul 20, 2022

Conversation

frostming
Copy link
Contributor

@frostming frostming commented Jul 20, 2022

The previous command doesn't clean the cache correctly. After this change, the lock time of poetry will become comparable. (9 seconds is too dramatic)

@ipmb
Copy link
Member

ipmb commented Jul 20, 2022

Thanks @frostming !

@ipmb ipmb merged commit 8aec527 into lincolnloop:main Jul 20, 2022
@frostming frostming deleted the patch-1 branch July 20, 2022 14:19
@frostming
Copy link
Contributor Author

Hi, do you have any idea why the site is not updated with the latest result: https://github.com/lincolnloop/python-package-manager-shootout/actions/runs/2708687120

@ipmb
Copy link
Member

ipmb commented Jul 21, 2022

It takes an average of the last 6 runs (which run every 6 hours) so it will take a while for the results to fully change. Since they are running on shared hardware, I wanted to avoid an anomaly skewing the results too much, but maybe 6 is excessive.

ipmb added a commit that referenced this pull request Jul 21, 2022
Changes would take to long to be reflected on the live site.
refs #4
ipmb added a commit that referenced this pull request Jul 21, 2022
Changes would take to long to be reflected on the live site.
refs #4
@ipmb
Copy link
Member

ipmb commented Jul 21, 2022

I dropped it down to 4 which should only include the changes here. You can see the aggregated results here: https://github.com/lincolnloop/python-package-manager-shootout/actions/runs/2712516720

It looks like poetry's slowest runs are pretty close to pdm's fastest runs which is why some individual results are very close, but there is still a gap on the aggregated average.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants