NimbusML icon indicating copy to clipboard operation
NimbusML copied to clipboard

NimbusML misidentifies ML.NET `FastTree` learner as `FastTrees`

Open TomFinley opened this issue 6 years ago • 1 comments

So in ML.NET, we have this class.

https://github.com/dotnet/machinelearning/blob/master/src/Microsoft.ML.FastTree/FastTreeClassification.cs

The corresponding NimbusML learner is called FastTrees.

So, whether or not this is a good name or not is not really my point, but rather:

  1. If we think that trees is a better name than tree, then this is stuff that really ought to be propagated to the ML.NET ecosystem.

  2. If the goal is consistency with sklearn vs. consistency with ML.NET this seems totally fine, but then why are we bothering with keeping part of what is the brand name, that is, FastTree? If the goal is to have the name be descriptive, I think incorporating parts of brand names from ML.NET can only add to confusion. (E.g., FastTree is so named, yet AFAIK in many situations LightGBM is faster.)

TomFinley avatar Oct 24 '18 20:10 TomFinley

I think it's worth having a broader discussion regarding alignment with ML.NET (some other terms don't match as well).

GalOshri avatar Oct 25 '18 00:10 GalOshri