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

Check the Tucker branch's metric values #9

Closed
willpearse opened this issue Aug 22, 2014 · 3 comments
Closed

Check the Tucker branch's metric values #9

willpearse opened this issue Aug 22, 2014 · 3 comments
Assignees

Comments

@willpearse
Copy link
Owner

Please have a quick final test of the code. I think it works, but you (Caroline) are the one who knows best :D

@cmtucker
Copy link
Collaborator

Sure, will do in the a.m.

On Aug 21, 2014, at 8:35 PM, Will Pearse notifications@github.com wrote:

Please have a quick final test of the code. I think it works, but you (Caroline) are the one who knows best :D


Reply to this email directly or view it on GitHub.

@cmtucker
Copy link
Collaborator

All the ecoPD code looks fantastic, I ran it to confirm the values match the original data, and they match, no problems.
I noticed that in the evenness.R code (.scheiner function) that the abund=FALSE option fails for me. It appears to do so at L402 ("data$comm <- as.numeric(data$comm > 0)") since this causes the next line 405 (output <- numeric(nrow(data$comm)) ) to produce an error, since you have lost the matrix structure for data$comm.
I won't edit the code, in case there's a particular way you want to adjust it, but if you can't replicate the error, let me know.

@willpearse
Copy link
Owner Author

Whooops. This is why I shouldn't write code late at night! Thanks Caroline, I'll get on that...

willpearse added a commit that referenced this issue Aug 22, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants