Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Time varying intercept #628
Time varying intercept #628
Changes from 40 commits
faaba0f
3ce5ac4
dac69da
ec316b4
7e8aee7
b0aaa2d
16f9414
8ab0532
c06c06d
b70cd06
9b1287b
e01f9ac
59708cf
83e7103
196d720
c77ddce
f25bc6e
1b3e73e
32f11e1
b35df87
293e752
910d223
15308ab
19d13d8
0fad32b
973a921
95c7ee8
4158768
843ec21
1c90255
d5e1699
74c09c0
a6c5972
374303c
3ac1f6a
0bfbbe4
665d1d2
6defce8
5f5be67
33f8f7b
7677015
e0b8ad6
3ac0585
3d207b2
ba1de7b
85cf5aa
0f89bd4
23ea9ec
d4688be
d3ced36
f8c0b3c
3f88589
f0e59e6
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Check warning on line 374 in pymc_marketing/mmm/base.py
Codecov / codecov/patch
pymc_marketing/mmm/base.py#L374
Check warning on line 468 in pymc_marketing/mmm/base.py
Codecov / codecov/patch
pymc_marketing/mmm/base.py#L468
Check warning on line 470 in pymc_marketing/mmm/base.py
Codecov / codecov/patch
pymc_marketing/mmm/base.py#L470
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe
time_index
should bemutable=True
no?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do we expect to happen with these
None
defaults?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's are checks starting on like 399, which set them if they are None. The reason that we wan't directly set them here is that the best defaults are estimated from the data, and we can't know here.
Check warning on line 744 in pymc_marketing/mmm/delayed_saturated_mmm.py
Codecov / codecov/patch
pymc_marketing/mmm/delayed_saturated_mmm.py#L744