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

metaMDS(..., engine ="isoMDS") produces error #310

Closed
martinmodrak opened this issue Apr 5, 2019 · 4 comments
Closed

metaMDS(..., engine ="isoMDS") produces error #310

martinmodrak opened this issue Apr 5, 2019 · 4 comments
Labels
bug

Comments

@martinmodrak
Copy link

@martinmodrak martinmodrak commented Apr 5, 2019

I have frequent convergence problems with the default setting of metaMDS so I wanted to try isoMDS instead of the default, but the code does not complete. Reproducible example follows:

library(vegan)
library(MASS)
data(varespec)
metaMDS(varespec)  #Completes, but does not converge
metaMDS(varespec, engine = "isoMDS")  #Does not complete

The error from the last line is

Error in out$dist/scl : non-numeric argument to binary operator

I am using vegan 2.5-4 with R 3.5.3 on Windows

@jarioksa jarioksa added the bug label Apr 5, 2019
@jarioksa
Copy link
Contributor

@jarioksa jarioksa commented Apr 5, 2019

Yes, this is a bug: isoMDS does not return information you need for this scaling. However, you really should not use isoMDS – it has several problems. However, I'll fix this.

@martinmodrak
Copy link
Author

@martinmodrak martinmodrak commented Apr 5, 2019

  1. Thanks for the quick reaction. 2) I understand there are issues with isoMDS, I was mostly just curious what would happen 3) Sorry if I sounded like demanding support, I definitely don't need this to be fixed to carry on with life.

Thanks for the nice package!

@jarioksa
Copy link
Contributor

@jarioksa jarioksa commented Apr 5, 2019

Don't worry: we think that if we provide an option like engine the option should work! I am not offended or disturbed of reports of failure, but I'll do my best to fix the problems, and I'll fix this one. We just did not have a test to the isoMDS engine and this fault slipped through when we made an unrelated change.

About your actual problem of no-convergence: Have you read the section "Convergence Problems" in the metaMDS help (?metaMDS), and followed those procedures?

jarioksa added a commit that referenced this issue Apr 6, 2019
jarioksa added a commit that referenced this issue Apr 6, 2019
(cherry picked from commit 8ad3e80)
@jarioksa
Copy link
Contributor

@jarioksa jarioksa commented Apr 6, 2019

Fixed with commit 8ad3e80

@jarioksa jarioksa closed this Apr 6, 2019
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.