cositools / cosipy

The COSI high-level data analysis tools
Apache License 2.0
3 stars 16 forks source link

Fix documentation link and only print one log-likelihood warning #151

Closed eneights closed 4 months ago

eneights commented 4 months ago

Updated index.rst to fix link to GRB spectral fitting notebook.

Updated COSILike and spectral fitting notebooks to only print one warning about the log-likelihood per fit. This is to avoid having the same warning printed repeatedly every time the log-likelihood is calculated in the example notebooks and documentation.

ckarwin commented 4 months ago

Very nice @eneights! I ran through the GRB notebook and it works well. I also applied these changes to the extended source NB, in order to prevent unwanted output (Eliza added this to her PR). Note that for the extended source NB with multiple sources, there is a bunch of additional output from 3ML pertaining to the updated log values at each iteration of the fit. In order to silence this I also used the following right before the fit cell:

from threeML import update_logging_level
update_logging_level("CRITICAL")

Now the spectral fit NBs no longer have tons of output from the fit, which will make the tutorials much nicer and easier to read.