ignite-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [ignite] dehasi commented on issue #6567: IGNITE-9978 Implement Compound Naive Bayes classifier
Date Wed, 05 Jun 2019 21:55:52 GMT
dehasi commented on issue #6567: IGNITE-9978 Implement Compound Naive Bayes classifier
URL: https://github.com/apache/ignite/pull/6567#issuecomment-499270291
 
 
   @avplatonov Thank you for the feedback. 
   I moved feature filtering to Compound Bayes.  Not sure if my approach is ok or you want
a separate preprocessor? 
   
   Regarding vector-projection, I like the idea itself and if we need this structure I'll
be happy to implement it.
   
   If I need to enrich other Bayesian models with `probabilityPowers` method, does it make
sense to create a separate interface for it? Something like `CompoundBayesCompatibleModel`
and force people to implement this method in (probably) some other types of Bayesian algorithms.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message