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: Allow in- and out-of-sample parameters to be set #388

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bashtage
Copy link
Owner

Separate setting of in- and out-of-sample parameters in forecast

[skip ci]

Separate setting of in- and out-of-sample parameters in forecast

[skip ci]
@pep8speaks
Copy link

Hello @bashtage! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 109:17: E126 continuation line over-indented for hanging indent
Line 112:17: E126 continuation line over-indented for hanging indent
Line 1195:62: W504 line break after binary operator
Line 1199:65: W504 line break after binary operator
Line 1510:70: W504 line break after binary operator

Line 330:60: W504 line break after binary operator
Line 679:47: E126 continuation line over-indented for hanging indent

Line 276:74: W291 trailing whitespace
Line 277:70: W291 trailing whitespace
Line 311:74: W291 trailing whitespace
Line 312:70: W291 trailing whitespace
Line 352:74: W291 trailing whitespace
Line 353:70: W291 trailing whitespace
Line 577:72: W291 trailing whitespace
Line 969:30: E126 continuation line over-indented for hanging indent
Line 1052:100: E501 line too long (112 > 99 characters)
Line 2072:73: W504 line break after binary operator
Line 2735:65: E126 continuation line over-indented for hanging indent

Base automatically changed from master to main February 22, 2021 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants