-
Notifications
You must be signed in to change notification settings - Fork 10
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
bitc_util.py not writing out all statistics #58
Comments
H_0 and sigma are in units of calorie instead of microcalorie. Fixing it together with the newly implemented models in PR #57. Labeling in bitc_util.py is also inconsistent and wrong. |
FYI the calculation in models.py looks correct, but order of magnitude used is just troublesome. In my earlier efforts to clean up the unit system, the order of magnitude was changed. |
This looks better. Adding comments to the models code where order of magnitude conversions are taking place between units , e.g. in the expected_injection_heats functions that do not have explicit units on the inside.
|
This is looking good! I think we should write a method to correctly output means and standard errors to correct sig figs. We assume we only know the standard error to one sig fig, so we truncate it to this sig fig. This determines how many decimal places of the corresponding quantity to print. For example, this would become
|
Good to know. Are there any existing (python) tools to do smart-formatting like that? I could probably write something myself if necessary.
Absolutely! Will see what I can do there. Maybe pymc.matplot already has some functionality. |
Great find, it works well as far as I can tell. Should I add it to the bayesian-itc repo? I might make a separate conda package for it sometime. |
Found this on pypi
Has the same name, provides a similar functionality. Might be better to use this instead of the other script just to prevent confusion, though I prefer the documentation on the carpy one. Does the GPL allow redistribution under a different name? |
Sounds good! |
Some values not properly extracted, returns 0.
The text was updated successfully, but these errors were encountered: