Open eclake opened 3 years ago
hi @eclake, the first error seems to be related to a new default option of pyplot.subplots
squeeze : bool, default: True
Your solution will break the code when n_outer and n_ranges are > 1, so the best option is probably to set explicitly squeeze=False
in the plt.subplots calls in these lines, can you perhaps test it?
The solution to the second issue seems fine ! :) I'll take a look at pyp-beagle right now, I'd like to update it to Python 3, and in the meantime I can integrate these changes...
OK, it looks like the changes I suggested that could break things got included when my calibration axes plots got added... This means the newest version of pyp_beagle is working for me, but might break in other instances as @jacopo-chevallard mentioned earlier. I will see if I can improve this
Two errors when using --show-residual for plotting the marginal for a full spectroscopic fit
First error:
Which I managed to fix by changing these lines to read
instead.
When I was trying to add the residual to the output spectrum marginal plot, I got the following error
If I changed these lines to read
instead, it worked.
The example I was running pyp-beagle on is on neogal at /mnt/globalNS/tmp/curtis/BEAGLE_workdir/results/NIRCam_NIRSpec_data_challenge_1/fit_R100_photom_poly_1_7340_test/
Happy to put together a pull request for these fixes if they look right!