-
-
Notifications
You must be signed in to change notification settings - Fork 26.6k
π π€ CI Update lock files for main CI build(s) π π€ #31918
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
π π€ CI Update lock files for main CI build(s) π π€ #31918
Conversation
64f1a96 to
dc9c6ad
Compare
|
Looks like OpenML has issues. Its status page is all green but I am getting a 404 when I tried to download a dataset e.g. clicking the download button from this page For further reference the This may have been noticed by others, see openml/OpenML#1232 and openml/openml-python#1433 that seemed similar and recent. |
|
Unfortunately this seems like an OpenML issue that is a bit out of our hands ... hopefully it will be fixed in the upcoming days. Would that be too inconvenient for you to work on your PRs that need the latest Cython with the lock-files updated, wait for reviews, and then once your PRs are reviewed hopefully the OpenML issue would have been fixed? |
|
Let's say I can live with it. |
|
The OpenML issue should be fixed now :). |
|
Thanks a lot @joaquinvanschoren for the fix! I relaunched the doc build again and it looks like we still have one issue that may be unrelated to the ARFF download issue that you fixed. It seems like we are not able to download a parquet dataset that one of our example uses: Edit: I actually tried to wait for 5 minutes with I saw in openml/openml-python#1433 that there may have been some issues about slow downloads recently as well. |
Maybe a comment to see is this one? openml/OpenML#1232 (comment) |
dc9c6ad to
7737ae5
Compare
|
This is also fixed since last weekend. |
|
Thanks, I can confirm that now downloading the problematic The weird thing is that one of the doc CI build failed this morning (Monday August 18, ~5am UTC) with I relaunched the build to see if the problem goes away. Note: |
|
So it seems like the doc build succeeded this time π! |
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.
LGTM, let's merge this one!
Note: for some reason the doc and doc-min-dependencies switched from OpenBLAS to MKL, but I think that's fine.
Co-authored-by: Lock file bot <[email protected]>
Co-authored-by: Lock file bot <[email protected]>
Update lock files.
Note
If the CI tasks fail, create a new branch based on this PR and add the required fixes to that branch.