I would rename "tmp" as "work_buffer". Same for "VI" I don't understand the meaning either.
I would also like to avoid "p" in the public API of kNN methods. Maybe use a "distance" parameter that could accept string values such as distance="euclidean" or distance="manhattan" or a float value that would be interpreted as an arbitrary Minkowski distances. -- Olivier ------------------------------------------------------------------------------ Ridiculously easy VDI. With Citrix VDI-in-a-Box, you don't need a complex infrastructure or vast IT resources to deliver seamless, secure access to virtual desktops. With this all-in-one solution, easily deploy virtual desktops for less than the cost of PCs and save 60% on VDI infrastructure costs. Try it free! http://p.sf.net/sfu/Citrix-VDIinabox _______________________________________________ Scikit-learn-general mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/scikit-learn-general
