NIFTy-PPL / NIFTy

Probabilistic programming framework for signal inference algorithms that operate regardless of the underlying grids and their resolutions

https://ift.pages.mpcdf.de/nifty/index.html

GNU General Public License v3.0

13 stars 2 forks source link

Plot requires labels in `demos/0_intro.py` #16

Closed apizzuto closed 2 months ago

apizzuto commented 2 months ago

Part of https://github.com/openjournals/joss-reviews/issues/6593

Just a tiny nit, but can you please label the Ax1 subplot in the 0_intro.py demo? Looks like the values correspond to (cfm.amplitude(pos_truth)[1:], post_a_mean) but this isn't obvious when looking at the figure.

Edenhofer commented 2 months ago

Thanks! Right, yeah it is definitely not obvious what this plot shows :see_no_evil:

21 adds labels and tweaks the other plots some more too.