-
Notifications
You must be signed in to change notification settings - Fork 11
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
Multiple improvements to pymcmodels #83
Merged
Merged
Changes from 13 commits
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
f5b5f51
Add support for specified F_PL and its uncertainty dF_PL
jchodera 0bdb607
Remove 'normal' concentration priors; use LogNormalWrapper to simplif…
jchodera d17a381
Clean up LogNormalWrapper
jchodera 4814199
Convert quantum yield priors to lognormal by default.
jchodera 0d3e197
Convert fluorescence and absorbance errors to lognormal
jchodera 2bc6031
Fix py3 issues in quickmodel
jchodera 8c89cd6
Fix python 3 incompatibility in platereader.py
jchodera c7697ad
Have quickmodel print exceptions it runs into
jchodera 99a0839
Adjust Metropolis step methods for new LogNormal priors; some py3 fixes
jchodera 85e15da
Start [L]=0 implementation
jchodera cf69d59
Merge branch 'improvements' of github.com:choderalab/assaytools into …
jchodera 3e99202
Add quickmodel to continuous integration tests.
jchodera fd9724e
Allow ligand or protein concentrations to be zero.
jchodera b3a6dcc
Add command-line control over number of samples and thinning for quic…
jchodera e1f738a
Restore close to previous defaults for quickmodel
jchodera b1026a4
Add pymbar to requirements
jchodera f26e8c0
Whoops! Previous behavior was actually 10000 samples, not 1000
jchodera ef095fe
quickmodel now plots equilibrated traces in a different shade; fix bu…
jchodera f41dc66
Fixed bug where same ligand concentration was being used for both +/-…
jchodera 20a3e81
Decouple +/- protein buffer background priors
jchodera dbec19f
Restored Metropolis step methods with better step size guesses
jchodera b02d9ac
Add AdaptiveMetropolis for correlated DeltaG and log_F_PL moves
jchodera d841d8f
Added tune_throughout=True
jchodera fcde1cf
Allow Metropolis methods to tune throughout
jchodera 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 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 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 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 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
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains 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
131 changes: 65 additions & 66 deletions
131
examples/direct-fluorescence-assay/3a Bayesian fit xml file - SrcGefitinib.ipynb
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains 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 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 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
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.
It's great that
quickmodel
has been added to travis. However,quickmodel
runs the default number of MCMC steps, currently set as 20000 PyMC moves, which may take a while on travis. How about we augment theargparser
onquickmodel
's so that we can specify far fewer moves on travis? Parsing the number of MCMC moves toquickmodel
will make it easier to use as well.