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
misc, tests, diagnostic #997
Conversation
rebased on master added some delinting |
@jseabold This needs to go into 0.5.0 |
Ok. Will wait for Travis. Let me know if/when to merge. |
if alternative in ['two-sided', '2s', 'smaller']: | ||
crit_low = stats.t.ppf(alpha_, df) | ||
pow_ += stats.nct._cdf(crit_low, df, d*np.sqrt(nobs)) | ||
#print crit_low, df, d*np.sqrt(nobs) | ||
if np.isnan(crit_upp): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
crit_low?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
copy-paste error and not running the tests
also missing an any()
fixed
Merge? |
go ahead, all green (I also ran the tests this time with the oldest versions.) |
TST/BUG: Bug fixes and tests for diagnostic and pylint fixes.
TST/BUG: Bug fixes and tests for diagnostic and pylint fixes.
issue #962: add tests, surprisingly all tests pass without changes to the functions even though they use wrapped results instead of
._results
(tested only with pandas 0.7.3 so far)