DOC cleanup the roadmap #15332
DOC cleanup the roadmap #15332
Conversation
doc/roadmap.rst
Outdated
| * Learners directly handling missing data :issue:`13911` | ||
| * Making sure meta-estimators are lenient towards missing data, | ||
| :issue:`15319` | ||
| * Non-trivial imputers |
There was a problem hiding this comment.
I guess we can even remove it now unless there are other open issues/PR
There was a problem hiding this comment.
The iterative imputer still has a blocker issue.
doc/roadmap.rst
Outdated
| * Making sure meta-estimators are lenient towards missing data, | ||
| :issue:`15319` | ||
| * Non-trivial imputers | ||
| * Learners directly handling missing data |
There was a problem hiding this comment.
remove? Unless we want to support it for the tree module as well?
There was a problem hiding this comment.
That's what I was thinking when I didn't remove it. I also couldn't think of any other estimator which could easily support this.
| #. Improved tracking of fitting | ||
|
|
||
| * Verbose is not very friendly and should use a standard logging library | ||
| :issue:`6929` | ||
| :issue:`6929`, :issue:`78` | ||
| * Callbacks or a similar system would facilitate logging and early stopping | ||
|
|
||
| #. Distributed parallelism |
There was a problem hiding this comment.
need to skip line, rendering is wrong
| * Joblib can now plug onto several backends, some of them can distribute the | ||
| computation across computers | ||
| * However, we want to stay high level in scikit-learn | ||
| * Accept data which complies with ``__array_function__`` |
There was a problem hiding this comment.
I'm confused, isn't #15332 doing the exact opposite (i.e. not rely on __array_function__)?
There was a problem hiding this comment.
You mean Andy's PR? The idea is for now actively not to accept it, instead of giving odd errors if the user passes them, and then work towards possibly supporting them later.
There was a problem hiding this comment.
lol yeah sorry I meant #14702
|
#10463 has been addressed |
+1 |
|
I think I have applied the comments. |
doc/roadmap.rst
Outdated
| @@ -124,12 +135,12 @@ bottom. | |||
|
|
|||
| #. Better interfaces for interactive development | |||
|
|
|||
| * __repr__ and HTML visualisations of estimators :issue:`6323` | |||
| * |ss| __repr__ and HTML visualisations of estimators :issue:`6323` |se| | |||
There was a problem hiding this comment.
we've done repr but not HTML yet...
|
I think we should mention that the cross-out entries are "done" rather than "not relevant anymore". Also, not strong opinion but I find it hard to read full crossed-out sentences. I'd be happier with a section at the end where we put the currently achieved goals (each bullet point would need more context though). But again, no strong opinion. LGTM apart from the html thing mentioned by Joel |
|
Is this good now? |
|
Thanks @adrinjalali |
This cleans up the roadmap a bit. Removes the links to issues and PRs which are already solved, and is an effort to keep it up to date.
I'm happy to put things which are deleted in this PR back. I've mostly deleted them for us to talk about them if you still think they should be there.
ping @scikit-learn/core-devs