-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
DOC Fix doc build now that OpenML is back #30715
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
Conversation
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.
Thanks, @lesteve. Will add an inline comment with a TODO and merge tot get green CI.
@PGijsbers it looks like the equivalent I was wondering whether you would recommend relying directly on this URL though. My understanding is that this may be an OpenML implementation and that the parquet URL could well change in the future. The "right" way is to look at the data description URL and its
If that is considered too likely to change, maybe we could stick with the parquet file in our https://github.com/scikit-learn/examples-data repo as we are currently doing. For the context: this example is trying to be somewhat more realistic by showing how to load a parquet file directly. In the majority of cases, users are expected to load their own data rather than using |
OpenML should be mostly back to normal according to #30708 (comment).
Actually the example with the OpenML parquet file still fails. For now I am using https://github.com/scikit-learn/examples-data to host the file. In the medium term we will be able to use a similar URL in
https://data.openml.org
, see see #30708 (comment).