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

Remove dependence on Statistics or Finance toolbox #52

Open
peclayson opened this issue Oct 11, 2018 · 1 comment
Open

Remove dependence on Statistics or Finance toolbox #52

peclayson opened this issue Oct 11, 2018 · 1 comment

Comments

@peclayson
Copy link

Hi,

I noticed that in StanFit.m, the nansum function is used on lines 272 and 366. nansum is a function in the Statistics toolbox and in the finance toolbox.

nansum can be replaced with sum(x,'omitnan'), which would circumvent the user needing the two abovementioned toolboxes and serves the same purpose as nansum. I can't find any other functions that require the toolboxes in the MatlabStan scripts.

I know most people probably have one of these toolboxes, but it might make life easier on a few.

Hope it helps :)
Peter

@JoKeyser
Copy link

This issue has also been raised in issue #48 .

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