-
Notifications
You must be signed in to change notification settings - Fork 72
API change for the SyntheticControl
experiment class
#460
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
Open
drbenvincent
wants to merge
31
commits into
main
Choose a base branch
from
sc-api-change
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
7bbff4f
initial efforts
drbenvincent 7ece785
remove print statement
drbenvincent 98127fd
obs_indx -> obs_ind (see #459)
drbenvincent 82d041d
update API in tests for SyntheticControl class
drbenvincent 3bbabee
Merge branch 'main' into sc-api-change
drbenvincent 1f4d17e
tidy up + fixes
drbenvincent 182aac0
get deprecation tests working again
drbenvincent 876c154
bug fixes
drbenvincent 3d29fef
fix bug with SyntheticControl.get_plot_data_bayesian
drbenvincent 3eb24a9
use new API in scikit-learn integration test
drbenvincent bd9beaa
update the pymc synthetic control notebooks
drbenvincent ed62f0a
remove test_api_stability
drbenvincent a148ec3
fix bugs
drbenvincent 15454b2
bug fixing
drbenvincent b77c3f0
remove api backward compatibility and deprecation tests
drbenvincent c89a147
more deprecation removal
drbenvincent fad78d8
add additional asserts to integration tests to detect shape problems
drbenvincent 2137091
remove asserts which weren't doing the job I intended
drbenvincent 45f1b1a
start embracing xarray to handle broadcasting
drbenvincent 2726484
formatting
drbenvincent b920207
store data in xarray objects in more experiments
drbenvincent a28c5da
attempt to make LinearRegression doctest pass
drbenvincent 642f651
revert a change which seems no longer required
drbenvincent a17f5c9
fix some failing tests
drbenvincent 9941c02
all tests now passing 🎉 (one failing doctest)
drbenvincent 6db026e
update api calls in multi cell geolift notebook
drbenvincent b49ed7e
undo plot colour change that I made when debugging
drbenvincent 1f753e9
final doctest now passes 😍
drbenvincent d855557
update notebook on scikit-learn synthetic control example
drbenvincent c92c117
rerun synthetic control notebook
drbenvincent 2fcb3ba
fix minor issue in multi-cell geolift notebook
drbenvincent File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
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.
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.
dims of X as coeffs? Should surely be covariates... coeffs is the output, and while they should be 1:1 naming is a little misleading no?