Skip to content

CAUS-2: write CausalImpact shock-based estimator#25

Open
ingrid88 wants to merge 1 commit intoakelleh:masterfrom
ingrid88:shock-estimator
Open

CAUS-2: write CausalImpact shock-based estimator#25
ingrid88 wants to merge 1 commit intoakelleh:masterfrom
ingrid88:shock-estimator

Conversation

@ingrid88
Copy link
Copy Markdown

@ingrid88 ingrid88 commented Nov 5, 2016

flake8 didn't like the empty spaces in the file after some of the lines.

@akelleh
Copy link
Copy Markdown
Owner

akelleh commented Nov 6, 2016

Looks good, Ingrid! Ask questions and make comments here, and we can get this thing in there. Try to keep each PR to under 200 lines ... if it's getting longer, we'll merge it in and start a new logical addition.

@akelleh akelleh changed the title added empty class CausalImpact CAUS-2: write CausalImpact shock-based estimator Nov 7, 2016
@akelleh
Copy link
Copy Markdown
Owner

akelleh commented Nov 11, 2016

Hey Ingrid! Just checking in! lmk if you have any questions, and don't forget to keep your PR up to date, even if it's a total work in progress!

@ingrid88
Copy link
Copy Markdown
Author

Yeah --- i plan on working on it this weekend! I just havent even started yet.

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