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

ENH: avoid hard-listed bandwidths -- use present dictionary (+typos fixed) #1671

Merged
merged 1 commit into from May 20, 2014

Conversation

Projects
None yet
3 participants
@yarikoptic
Copy link
Contributor

commented May 16, 2014

it seems to work if docstring is populated this way, but didn't check if sphinx doesn't mind. It allows to avoid duplication and make docstring more 'factual' (before was missing one of the bandwidths)

@josef-pkt

This comment has been minimized.

Copy link
Member

commented May 20, 2014

looks good to me, merging

josef-pkt added a commit that referenced this pull request May 20, 2014

Merge pull request #1671 from yarikoptic/enh/typos_and_bandwidths
DOC/ENH: avoid hard-listed bandwidths -- use present dictionary (+typos fixed)

@josef-pkt josef-pkt merged commit 6237755 into statsmodels:master May 20, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
@coveralls

This comment has been minimized.

Copy link

commented May 20, 2014

Coverage Status

Coverage increased (+0.0%) when pulling 3713f02 on yarikoptic:enh/typos_and_bandwidths into b472807 on statsmodels:master.

@josef-pkt josef-pkt added PR labels May 21, 2014

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

Merge pull request statsmodels#1671 from yarikoptic/enh/typos_and_ban…
…dwidths

DOC/ENH: avoid hard-listed bandwidths -- use present dictionary (+typos fixed)
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.