Hello

While working on matrix factorization with missing values
<https://github.com/scikit-learn/scikit-learn/pull/4237> I faced a circular
import issue (
https://travis-ci.org/scikit-learn/scikit-learn/jobs/50276638#L1362).

The problem is that I want to add a new imputer, which should reside in
.preprocessing.imputation and use factorizer from decomposition. As you can
see in Travis, this leads to circular import, so I should not import
decomposition in preprocessing. That means I need to move factorization
somewhere.

So, the question is: where can I move it? The factorization core belongs to
decomposition (NMF is already there). I was thinking about the "root"
(where lda, qda, cross_validation and others are located), but not sure if
polluting it is a good idea.
------------------------------------------------------------------------------
Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server
from Actuate! Instantly Supercharge Your Business Reports and Dashboards
with Interactivity, Sharing, Native Excel Exports, App Integration & more
Get technology previously reserved for billion-dollar corporations, FREE
http://pubads.g.doubleclick.net/gampad/clk?id=190641631&iu=/4140/ostg.clktrk
_______________________________________________
Scikit-learn-general mailing list
Scikit-learn-general@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/scikit-learn-general

Reply via email to