Closed chto closed 7 years ago
Hi, @drphilmarshall
In this branch, I add two notebooks which are Sigma_Intrinsic_Analysis.ipynb
and SLTimer_TDC2_NewWhitening.ipynb
.
In Sigma_Intrinsic_Analysis.ipynb
: I do an analysis on how sigma_intrisic affect the likelihood.
In SLTimer_TDC2_NewWhitening.ipynb
: I do the same analysis as in Sigma_Intrinsic_Analysis.ipynb
, but change the whitening method.
The likelihood for new whitening method is strange and I am still finding the reason. But I think it is not likely to be caused by some bugs in the code, after checking the code again and again.
Therefore, I think it is a good time to let you know.
If you have any suggestion or comments, please let me know. Thanks.
@chto Great work! It's hard to comment inline on notebooks, so I'll do it here instead.
Sigma_Intrinsic_Analysis.ipynb
seasonal=True
, so I wonder: can we compare the absolute value of the likelihood in this analysis with that in your other Sigma_Intrinsic_Analysis.ipynb
notebook, where seasonal=False
? I think we might be able to. With new whitening, the log L values are around +1000, whereas in the other notebook they are more like -300 - so it does seem like the new whitening is a better model. You should think about whether or not this is a fair comparison though.seasonal
is also the wrong approach?OK, all these commits were also in your issue/42/initialization
branch, which I just merged into issue/40/tdc2gatewaytest
. Can you please pull this branch from the base repo and check that everything looks right to be your current version? Any problems, please issue them. Thanks!
Coverage decreased (-2.4%) to 13.604% when pulling 3d529c2d25b9c8c079aac977c850c0ad4ebc2c12 on chto:add_sigma_intrinsic_and_new_whitening into f7f37a1a97cbd16e31363fc8baf076525c9bc280 on DarkEnergyScienceCollaboration:master.