Skip to content
This repository has been archived by the owner on May 25, 2024. It is now read-only.

Switch RVM support to EMRVC #8

Open
halsw opened this issue Sep 9, 2021 · 1 comment
Open

Switch RVM support to EMRVC #8

halsw opened this issue Sep 9, 2021 · 1 comment
Labels
enhancement New feature or request

Comments

@halsw
Copy link

halsw commented Sep 9, 2021

Hello, and congratulations for your work. I've used random forest and SVC on complex datasets and work like a charm, on the other hand I didn't manage to get either RVC(sklearn_bayes) working properly or port_rvm, so my suggestion is to switch RVM support to EMRVC(sklearn_rvm)

sklearn_bayes fails to install, probably because it supports an older version of sklearn. I've managed to install it by modifying the code to support sklearn 0.24 and manually compiling cython source files, but even then it does not work properly and the C++ models I've managed to produce all seem to have 8 relevance vectors. On the flip side sklearn_rvm works really well and it will make your work less sklearn version dependent.

@eloquentarduino eloquentarduino added the enhancement New feature or request label Sep 9, 2021
@eloquentarduino
Copy link
Owner

Hi @halsw, I had troubles installing sklearn_bayes after a while. The reason I chose it over sklearn_rvm was that it was much faster to train. But if it doesn't play well with sklearn, I will work on adding sklearn_rvm support too, so one can choose either of the two.

I marked this issue as "enhancement" for the library.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants