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

bind gaussian #2

Closed
wants to merge 1 commit into from
Closed

bind gaussian #2

wants to merge 1 commit into from

Conversation

zvictor
Copy link

@zvictor zvictor commented May 26, 2016

For some reason, when using the gaussian function in my Meteor environment it does not work because this === Window inside the gaussian method.

This PR should make it safe.

@coveralls
Copy link

coveralls commented May 26, 2016

Coverage Status

Coverage increased (+0.2%) to 75.0% when pulling 8bce755 on zvictor:patch-1 into 63db557 on Planeshifter:master.

@Planeshifter
Copy link
Owner

Thanks for filing this pull request! I published a new version of the module that should solve your problems when using it with Meteor. I switched out the gaussian module to my own implementation of the Gaussian pdf.

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

Successfully merging this pull request may close these issues.

None yet

3 participants