-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Harmonics search should give a more helpful message if frequency doesn't exist in data (e.g. a CLEANest value added to top-hits) #61
Comments
Additional detail from Brad:
|
From Brad:
|
Further comment from Brad:
|
I saw this error while using DC DFT standard scan to test the harmonic tolerance range - I've tracked it down to where the issue is coming from which plot's domain and range is being used to compare the harmonic frequency against the data frequencies. When the top hits plot is made active/top in line 190 of PeriodAnalysis2DResultDialog.java, it makes that data set active as well so if the harmonic didn't happen to be one of the top hits then it says the frequency wasn't found. What I'm still not certain of is the implication - are the values showing up in the harmonics list not correct and it should be just the top hits, or is there an issue with the way the plot/data set is made active and it should really be pointing at the same data that was used to create the harmonics list? Also this was just one case I had tested and doesn't match all the workflows Brad has mentioned so there might be more. |
It may help us reason about this if we look at a particular target and JD range from AID. Do you recall the target and JD range where you saw the error and the particular table/plot selection? |
This was just with the default R Lyr and min/max JD of 2458417.5/2459148.5 The 3rd and 4th harmonics had the error in the table below - there probably were more but those were the ones I was using to test. The 5th harmonic in the list below was one of the top hits so when clicked it just highlighted that data point in the plot as opposed to showing the error. |
With that JD range I get a top hit at 55.93. Odd. Still thinking about all this. Could be a tolerance based problem again here. |
@dbenn I just retried and got 55.65. Not checked why yet but another data point. |
@mdadighat Hmm. I just repeated with the JD range you specified above and still get a 55.93 top-hit. |
@dbenn I was able to get 55.93 if I removed the .5 from both JDs which doesn’t make much sense since I’m not sure that even affected the number of observations. |
@mdadighat Curioser and curiosier. :) Have to give this some thought. What series did you choose for DCDFT Standard Scan? Vis or V? |
@dbenn I used Vis for all. I'll get the same value testing multiple times, and then if I come back to it after working on something else I get something different, which doesn't make sense. The frequencies are the same (at least as far as I can tell), but the period tends to differ by a bit. I think it's still independent of the other issue where only the top hit values don't cause the original error since top hits is the active data set for the plot as opposed to the full one (not sure if I'm wording that right). Also have you seen where the harmonics dialog pops up twice each time? |
@mdadighat Non-reproducible is never fun. Agreed that it doesn't make sense. Yes, I have seen the harmonic dialog pop-up twice bug. Annoying isn't it? I thought it was one that I had in SourceForge tickets and was going to "move it over". This bug may have something to do with VStar's Observer pattern based message notifier registering more than once for the message type. |
Brad W reports:
The text was updated successfully, but these errors were encountered: