-
Notifications
You must be signed in to change notification settings - Fork 32
Power analysis #20
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
Merged
Merged
Power analysis #20
Changes from 60 commits
Commits
Show all changes
62 commits
Select commit
Hold shift + click to select a range
70904c3
Create power_ppi.py
Michael-Howes 12ece40
Update power_ppi.py
Michael-Howes 03199e9
create power analysis function mean estimation
Michael-Howes eb7c7bd
Update power_ppi.py
Michael-Howes c307a5a
Create power_analysis.ipynb
Michael-Howes 90bad83
update power analysis to return dict
Michael-Howes a27fb59
add ols power analysis
Michael-Howes 904f410
add ols example
Michael-Howes 4edb6b9
Update power_analysis.ipynb
Michael-Howes 42fb6f4
add logisitic regression power analysis
Michael-Howes faa8703
moved power analysis
Michael-Howes 00973f0
Update power_ppi.py
Michael-Howes 8139a8f
add poisson power
Michael-Howes 6eb4f2d
add Poisson examples
Michael-Howes 3b2bd86
added ppi_power for user supplied rho
Michael-Howes 88957c3
Update power_analysis.ipynb
Michael-Howes 6635528
update .gitignore
Michael-Howes a46b0b3
added n_max constraint
Michael-Howes d06f02d
add warning for cheapest pair
Michael-Howes 9c0c822
run notebook
Michael-Howes 4899ace
Update power_ppi.py
Michael-Howes 6d4a77d
Merge branch 'aangelopoulos:main' into main
Michael-Howes 7a4f608
update power analysis
Michael-Howes a923b87
update notebook
Michael-Howes 9289979
init notebook
davidbroska 51e4f3f
update power analysis notebook
Michael-Howes 06d72d4
Merge branch 'main' of https://github.com/Michael-Howes/ppi_py
Michael-Howes 0f1e951
add power tests
Michael-Howes 2b7fa2f
add moral machine data and notebook with AMCE function
davidbroska 6f75e73
move AMCE function to utils
davidbroska 797643f
Update test_power_analysis.py
Michael-Howes 357eae5
Update power_ppi.py
Michael-Howes 48385b1
Update power_analysis.ipynb
Michael-Howes 3e03df0
add seed value
Michael-Howes b2f04b2
reformatting with black
Michael-Howes d6b0056
reformat with black
Michael-Howes 56d8f06
Work on documentation
Michael-Howes 6f59ea3
update documentation
Michael-Howes 40753d8
fix typos
Michael-Howes 1296f02
small edits
davidbroska 29aec95
unify terminology
davidbroska 606d415
reformat with black
Michael-Howes 34833c4
reformat with black
Michael-Howes 9066071
remove moral machine notebook
Michael-Howes 244e4f6
remove moral machine function from utils
Michael-Howes d1703c0
update gitignore
Michael-Howes ddabddb
update .gitignore
Michael-Howes cbb0951
add example .gitignore
Michael-Howes 295cb82
remove moral machine function from utils
Michael-Howes 3b0387b
use safe_expit
Michael-Howes b820440
update references
Michael-Howes 21b8252
update power analysis
Michael-Howes 7c727d3
update power analysis example
Michael-Howes b342d32
update power analysis
Michael-Howes 425b798
update power analysis example
Michael-Howes 76d4b8c
rename power_ppi to ppi_power_analysis
Michael-Howes 46f16ab
reformat with black
Michael-Howes 775e35a
update power analysis to used desired effective sample size instead o…
Michael-Howes 0d412d7
update power analysis tests
Michael-Howes 37cb62f
reformat with black
Michael-Howes cbe850c
[black for formatting]
aangelopoulos 5b801f0
[minor] made a quick fix to the notebook and also added credit to the…
aangelopoulos 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
Large diffs are not rendered by default.
Oops, something went wrong.
Oops, something went wrong.
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.
Fixed two errors:
First,
np.concatdoes not exist. Made itnp.concatenate.Second, was getting a
nanerror. Addednp.nan_to_num. It is a kluge to make the notebook work. Please feel free to add a different fix if there's one @Michael-Howes .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.
Actually, reverted the second change
np.nan_to_numbecause it looks like it was caused by some corrupted data on my end. Re-downloading the dataset fixed it.