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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adjust tolerances for SPSA test. #97

Merged
merged 1 commit into from Mar 14, 2019

Conversation

@rcurtin
Copy link
Member

commented Mar 13, 2019

This should fail less than ~1 in 1000 times.

Thanks to @SuryodayBasak and @favre49 for pointing this out in IRC. 馃憤

Adjust tolerances for SPSA test.
This should fail less than ~1 in 1000 times.
@zoq

zoq approved these changes Mar 13, 2019

Copy link
Member

left a comment

Looks good to me.

@mlpack-bot

mlpack-bot bot approved these changes Mar 14, 2019

Copy link

left a comment

Second approval provided automatically after 24 hours. 馃憤

@zoq zoq merged commit 1e4117b into mlpack:master Mar 14, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can鈥檛 perform that action at this time.