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

DOC:ArmaProcess class documentation typo fix #4593

Merged
merged 1 commit into from May 15, 2018
Merged

DOC:ArmaProcess class documentation typo fix #4593

merged 1 commit into from May 15, 2018

Conversation

tadeze
Copy link
Contributor

@tadeze tadeze commented May 5, 2018

Fixed typo for variable name in the documentation example for ArimaProcess class.

@tadeze tadeze changed the title ArmaProcess class documentation typo fix DOC:ArmaProcess class documentation typo fix May 5, 2018
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.06%) to 82.811% when pulling 8beccc1 on tadeze:master into 3d87728 on statsmodels:master.

@codecov-io
Copy link

Codecov Report

Merging #4593 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #4593   +/-   ##
=======================================
  Coverage   80.26%   80.26%           
=======================================
  Files         564      564           
  Lines       85592    85592           
  Branches     9679     9679           
=======================================
  Hits        68702    68702           
+ Misses      14662    14661    -1     
- Partials     2228     2229    +1
Impacted Files Coverage Δ
statsmodels/tsa/arima_process.py 78.22% <ø> (ø) ⬆️
statsmodels/stats/descriptivestats.py 24.13% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3d87728...8beccc1. Read the comment docs.

@ChadFulton
Copy link
Member

LGTM, @josef-pkt is it safe to merge yet?

@josef-pkt
Copy link
Member

josef-pkt commented May 15, 2018

There is no restriction for merging into master for 0.10.
0.9.0 release is out and backporting goes to or will happen in maintenance if needed.

@ChadFulton
Copy link
Member

Thanks for letting me know @josef-pkt, and thank you @tadeze for the improvement!

@ChadFulton ChadFulton merged commit 9f79a1c into statsmodels:master May 15, 2018
@josef-pkt josef-pkt added this to the 0.10 milestone Sep 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants