Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork8.1k
Also upload a subset of nightly wheels#21637
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
ianhi commentedNov 15, 2021
ooooh or better would be something like |
QuLogic left a comment
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.
We should be uploading to SciPy's nightly channel instead. I think that's waiting on@tacaswell for some keys.
tacaswell commentedNov 16, 2021
Thank you for prodding me to make progress on this. Current state: I have sorted out who to ask and have (just) provided them with my anaconda.org user name. Once I have that set up I'll sort out which secrets I need to put where... |
greglucas commentedNov 16, 2021
@ianhi, I like how you're thinking with all these PRs :) I think you can add an @tacaswell, It looks like most other projects don't upload wheels every night, so you might want to check and see what an acceptable/preferred cadence is there too. |
ianhi commentedNov 16, 2021
Yeah, something like weekly would also be totally fine that's still much faster than the mpl release cadence :). It would be handy if it would be well documented when this upload happened so that cron jobs that depend on that upload can happen a bit after. e.g. this upload will always happen at ___ on sunday |
tacaswell commentedNov 17, 2021
Update: I have the login/access, but have not generated the secrets yet (on my to-do for tomorrow). Consensus is nightly is OK. |
tacaswell commentedNov 17, 2021
There is now a token |
tacaswell commentedNov 17, 2021
https://github.com/h5py/h5py/blob/master/ci/upload_wheels.sh <- this is how h5py is doing the upload. |
greglucas commentedNov 18, 2021
We should also probably add some documentation about where these wheels can be found somewhere too once we upload them. Then this PR will close#9994 |
jklymak commentedNov 22, 2021
@ianhi, feel free to pop out of draft when ready... |
matthewfeickert commentedMar 30, 2022
@ianhi 👋 Thanks for getting this PR going and for all the work that you've done on moving nightly wheels forward in general. 🙏 Now that@tacaswell has the I can fully appreciate that you most probably have other works that is more pressing at the moment. So I would be happy to help get a GHA workflow for this. (Apologies in advance to you and the maintainers for the noise if this is currently on hold for other reasons not documented in this PR.) |
ianhi commentedMar 30, 2022
I think this is on hold because I forgot about it and then was traveling when jody sent a reminder and so forgot once more :(
by all means have at it :) - I'm actually traveling again on Friday and we know how that impacted things last time.... |
matthewfeickert commentedMar 30, 2022
100% understand. Been there many many times. :)
SGTM. 👍 I'll try to get something that looks reasonable on a branch on my fork going and then discuss it back on Issue#21635 before adding something else into the mix. Safe travels! |
ianhi commentedMar 31, 2022
Closing in favor of#22733 |
PR Summary
Add a second artifact upload to the wheels that build on every merge to main. This is a subset of the
wheelsupload but should be significantly smaller to enable fast downloading for 3rd party tests in CI.Closes:#21635