-
Notifications
You must be signed in to change notification settings - Fork 155
Initial PR to add functionality for new DES-Dovekie release #465
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
Conversation
Initial commit for updated SN-likelihood for use in DES-Dovekie.
Added a new class to read new updated inverse covariance matrices for SN
|
Added many more necessary code changes ; will need to update the sn-data as well for some processing. the cosmo_tests for SN passes, but I'd like to just fit FLCDM cosmology with SN only to check I loaded everything in appropriately. |
|
sn data v1.8 merged |
|
Not sure how to restart the tests. I think with sn_data updated to reflect the new hubble diagram the automatic tests should pass... if they don't, I would like to figure out why so I can fix them ! |
|
You need to at least push update with new sn data version? |
|
I think the current sn_data is fine, but I was accidentally still pointing at v1.7 in After deleting stuff on my end, reinstalling, I found this fixed the AssertionError that the tests were seeing... fingers crossed that it did ! |
should be camb not classy
Good afternoon;
I've added a couple of initial functions to start the process of integrating the new SN likelihood and data for DES-Dovekie. Starting this PR to begin the discussions on what I'm missing/what needs to be added and changed.