Closed darthoctopus closed 4 years ago
Hi @darthoctopus
Thanks for pointing this one out. I think there are two things at play here that make this difficult. First is that it looks like the prior is still a little bit on the sparse side in this region. Second, the data SNR is really high (which is a good thing) but the modes are very narrow.
The code is finding a solution where is the mode line width is much higher than it should be. In fact is very much in tension with the prior, it's just that the data are so good that the prior is overwhelmed. This is sort of desired behaviour except the solution is the wrong one.
Suggested fix - beef up the prior in only the line width parameter. We need a penalty for the specific case that the line width becomes much larger than expected. I'll have a think about what distribution that should be.
I hacked together a simple additional penalty on the line width and things behave much better. My hack now forces a slight over estimate of the line width but at least I think I know what I'm working with here.
OK - this works better now. I'll make a PR for the changes. They might still need a little more thought.
I have merged in the changes to the treatment of an additional linewidth penalty. @darthoctopus I think this will fix your particular problem and I hope this will more generally fix this issue of bad solutions from the data overwhelming the prior. Please shout if you continue to have problems and thank you for the input!
I used the prior values (in the data file
reference_epsilon_values_do_not_change_this.csv
) for KIC 11717120 to attempt to identify modes and peakbag the KASOC short-cadence power spectrum. The relevant line from the CSV file follows:I created the star object as:
This fails quite spectacularly: