MacPython / openblas-libs

BSD 2-Clause "Simplified" License
9 stars 13 forks source link

do not upload tarballs, stop uploading wheels to multibuild-wheels-staging #164

Closed mattip closed 2 months ago

mattip commented 2 months ago

xref scientific-python/upload-nightly-action#85

We don't have any more consumers of the tarballs, and the wheels have proved sufficient.

cc @rgommers

mattip commented 2 months ago

Copyinh from a review comment, I think we should also encourage projects using https://anaconda.org/multibuild-wheels-staging to move to https://anaconda.org/scietific-python-nightly-wheels. The latter has a script to remove old artifacts.

rgommers commented 2 months ago

Copyinh from a review comment, I think we should also encourage projects using https://anaconda.org/multibuild-wheels-staging to move to https://anaconda.org/scietific-python-nightly-wheels. The latter has a script to remove old artifacts.

I would caution that there are downsides as well. Not only is it more work to move over than clean up manually once every 2 years, but it's easier for the release manager to make a mistake when they have to manually copy ~30 wheels from a bucket where new nightly wheels may get uploaded daily or even in parallel with the release wheels. It's easy to miss one in that setup. I vote for doing nothing.

mattip commented 2 months ago

As far as I can tell, both scipy and numpy use the same workflow as this repo: the wheels are downloaded via tools/download-wheels.py (for scipy) and tools/download-wheels.py (for numpy). So it should be simple to change those scripts when next someone cares enough about it.

mattip commented 2 months ago

I will merge this to no longer upload the unused tarballs.

rgommers commented 2 months ago

Ah fair enough, forgot about download-wheels.py. That should be okay then.