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
sndfile-spectrogram produces poor spectrograms for some inputs. #15
Comments
Thanks. Applied. |
This issue is still with us unfortunately. Note: the above interpolate from spec_len==2880 to heignt==480. However, the results are just as bad at size 480x2880 so it doesn't look like the interpolator. I'd like to get to the bottom of this. Erik, can you reopen this issue please? |
The is an extremely artificial signal. I think what you are seeing is the effect of the transient at the onset of sine tone. |
I don't think it's the onset (which is brusque, yes) because the erroneous vertical lines only appear when the FFT window only covers an area with pure 50Hz sine tone. It's only when a constant 50Hz tone is the only signal in town that the effect occurs, the same as with the "chirp" test piece. A gut felling says "integer overflow", but it can't be because we are in floating point world! |
For the 50Hz signal, I wonder is the window length is too small to capture a full cycle of the sine wave. |
A good experiment would be to do exactly this test for something around 200Hz> If there is no problem there, then reduce the frequency until it becomes a problem. That might give us an idea what |
Ok, I'm going to break the |
Martin Guy did a lot of investigative work that showed that the colour map calculation was producing a bunch of strange artifacts. #15 The simple fix here seems to fix at least some of those.
Martin Guy did a lot of investigative work that showed that the colour map calculation was producing a bunch of strange artifacts. #15 The simple fix here seems to fix at least some of those.
Please test commit:
I think that may have fixed it. |
Yes, that fixes all the above testcases. |
Wonderful! |
For instance this pair of commands produces a spectrogram with a bunch of rubbish in it.
The text was updated successfully, but these errors were encountered: