You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We are trialling starfish to see if we can implement some of your ideas here into our groups existing radiative transfer work.
We have your example files (setup.ipynb/single.ipynb) and it all runs smoothly apart from the sigma-contours are enormous (1e8) on the residuals plot. Example below.
We have managed to reduce this error through changing the log_amp parameter in the SpectrumModel(...) function to -11 (instead of 38) and changing the priors "global_cov:log_amp" value to (-11, 3) instead of (38,1). For us, this produces a similar looking plot to the readthedocs example. Numbers and plot below
Is a log_amp value of -11 considered to be okay/reasonable and normal behaviour?
As this seems significantly different to the original.
I'm not sure if you have changed something in the code since the readthedocs was written that might account for this difference?
The only other difference we had noted was the numbers outputted from training the emulator was different, such as the log likelihood. Does this all sound okay? Numbers from initial and secondary training below.
1st
2nd
This was verified by my supervisor on a separate machine. We both produced identical log-likelihood numbers of the emulator training session and in turn, identical log_amp numbers. Starfish version 0.4.2
Many thanks,
Austen
The text was updated successfully, but these errors were encountered:
It's been a while since I've used Starfish to fit spectra (~3-4 years) and I only used it on a couple stars from the same low resolution instrument (IRTF/SPeX) so I'm actually not as familiar with reasonable outputs. That being said, if the fitting looks good (it does) and you're not having troubles with convergence (looks okay since two separate computers got same result) then I wouldn't be too wary of it.
To add to this thread, the issue with the disparity seems to be coming from this commit, where planck scaling was removed from the log_amp parameter: 3f058df
Hi,
We are trialling starfish to see if we can implement some of your ideas here into our groups existing radiative transfer work.
We have your example files (setup.ipynb/single.ipynb) and it all runs smoothly apart from the sigma-contours are enormous (1e8) on the residuals plot. Example below.
We have managed to reduce this error through changing the log_amp parameter in the SpectrumModel(...) function to -11 (instead of 38) and changing the priors "global_cov:log_amp" value to (-11, 3) instead of (38,1). For us, this produces a similar looking plot to the readthedocs example. Numbers and plot below
Is a log_amp value of -11 considered to be okay/reasonable and normal behaviour?
As this seems significantly different to the original.
I'm not sure if you have changed something in the code since the readthedocs was written that might account for this difference?
The only other difference we had noted was the numbers outputted from training the emulator was different, such as the log likelihood. Does this all sound okay? Numbers from initial and secondary training below.
1st
2nd
This was verified by my supervisor on a separate machine. We both produced identical log-likelihood numbers of the emulator training session and in turn, identical log_amp numbers. Starfish version 0.4.2
Many thanks,
Austen
The text was updated successfully, but these errors were encountered: