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

Better handle missing benchmarks when calculating beta #859

Merged
merged 3 commits into from Nov 19, 2015
Merged

Conversation

@richafrank
Copy link
Member

@richafrank richafrank commented Nov 18, 2015

Return NaN beta when missing benchmarks instead of raising LinAlgError

@richafrank
Copy link
Member Author

@richafrank richafrank commented Nov 18, 2015

For #851

@richafrank richafrank force-pushed the future_beta branch from 7d872ea to 4237a08 Nov 19, 2015
@richafrank
Copy link
Member Author

@richafrank richafrank commented Nov 19, 2015

@twiecki look ok to you?

@twiecki
Copy link
Contributor

@twiecki twiecki commented Nov 19, 2015

LGTM /signoff.

@richafrank
Copy link
Member Author

@richafrank richafrank commented Nov 19, 2015

Thanks!

richafrank added a commit that referenced this pull request Nov 19, 2015
Better handle missing benchmarks when calculating beta
@richafrank richafrank merged commit dc68ad1 into master Nov 19, 2015
2 checks passed
2 checks passed
Scrutinizer 1 updated code elements
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@richafrank richafrank deleted the future_beta branch Nov 19, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.