Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mlp model with RSNNS package masks confusionMatrix, train #256

Closed
tobigithub opened this issue Oct 3, 2015 · 3 comments
Closed

mlp model with RSNNS package masks confusionMatrix, train #256

tobigithub opened this issue Oct 3, 2015 · 3 comments
Labels

Comments

@tobigithub
Copy link

@tobigithub tobigithub commented Oct 3, 2015

HI,
downloading the test or example data from > model <- "mlp" the following issue occurs:

> library(RSNNS)
Loading required package: Rcpp

Attaching package:RSNNSThe following objects are masked frompackage:caret:

    confusionMatrix, train

this will lead from the correct

> apropos("confusion")
[1] "confusionMatrix"       "confusionMatrix.train"

to the incorrect:

> apropos("confusion")
[1] "confusionMatrix"       "confusionMatrix"       "confusionMatrix.train"

This will further down the road lead to the following error for all packages that are used after the "mlp" training model such as

Error in confusionMatrix :  $ operator is invalid for atomic vectors
or
Error in toNumericClassLabels(targets) :    (list) object cannot be coerced to type 'double'

Still need to verify that but it is under RSNNS_0.4-7, Rcpp_0.12.0 , caret_6.0-52
under PRO 3.2.2 under Windows. Basically, code that was running OK, will after the install of the mlp training model, generate only errors such as confusionMatrix $ operator is invalid for atomic vectors or others. One solution would be to use:

caret::confusionMatrix(...)

But I have not tested that for all scenarios.
Cheers
Tobias

topepo added a commit that referenced this issue Oct 3, 2015
@topepo
Copy link
Owner

@topepo topepo commented Oct 3, 2015

I checked in a change. Please give it a try to see if it fixes the issue for you.

Thanks,

Max

@topepo topepo added the next release label Oct 3, 2015
@tobigithub
Copy link
Author

@tobigithub tobigithub commented Oct 4, 2015

Cool. Thank you
Tobias

@topepo
Copy link
Owner

@topepo topepo commented Oct 11, 2015

Looking at my changes, there really isn't anything else that I can do. The function in RSNNS isn't object oriented so that does not solve it. You will have to call using caret::confusionMatrix to get the right version when you have them both loaded.

@topepo topepo closed this Oct 11, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.