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

circ_vmrnd #7

Open
philippberens opened this issue Jan 26, 2017 · 1 comment
Open

circ_vmrnd #7

philippberens opened this issue Jan 26, 2017 · 1 comment

Comments

@philippberens
Copy link
Contributor

I found a small inconsistency in the circ_vmrnd program. On line 50, it should read:

alpha = 2pirand(n,1)-pi;

Currently the "-pi" isn't in the code, which means for large values of kappa, the distribution takes values on the interval (-pi,pi), while for kappa close to 0 it takes values between (0,2pi).

adehad added a commit to adehad/circstat-matlab that referenced this issue May 21, 2019
@jCalderTravis
Copy link

In this case of small kappa, the code also does not reshape the output. In contrast, usually when the input argument, n, is a two-element vector the output is reshaped. Shall I change this and submit a pull request?

Thank you for an amazing toolbox.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants