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

STY: Pep 8 cleanup #1368

Merged
merged 1 commit into from Feb 7, 2014

Conversation

Projects
None yet
3 participants
@jseabold
Copy link
Member

commented Feb 6, 2014

More pep-8.

# ('Prob (F-statistic):', ["%#6.3g" % self.f_pvalue]),
('Log-Likelihood:', None), # ["%#6.4g" % self.llf]),
('AIC:', ["%#8.4g" % self.aic]),
('BIC:', ["%#8.4g" % self.bic])

This comment has been minimized.

Copy link
@josef-pkt

josef-pkt Feb 7, 2014

Member

there is now an extra whitespace after the opening [

This comment has been minimized.

Copy link
@jseabold

jseabold Feb 7, 2014

Author Member

Fixed. My pep-8 checker doesn't complain about this.

@josef-pkt

This comment has been minimized.

Copy link
Member

commented Feb 7, 2014

looks good to me
AFAIK, we don't have any PR's where this would cause merge conflicts.

@coveralls

This comment has been minimized.

Copy link

commented Feb 7, 2014

Coverage Status

Coverage remained the same when pulling ba44f8f on jseabold:model-pep8 into a80dacb on statsmodels:master.

jseabold added a commit that referenced this pull request Feb 7, 2014

@jseabold jseabold merged commit 4ef7fbb into statsmodels:master Feb 7, 2014

1 check passed

default The Travis CI build passed
Details

@jseabold jseabold deleted the jseabold:model-pep8 branch Feb 7, 2014

PierreBdR pushed a commit to PierreBdR/statsmodels that referenced this pull request Sep 2, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.