Skip to content
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

Quetion on Signal Gen Documentation on Wiki #1139

Closed
damercer opened this issue Jul 1, 2021 · 3 comments
Closed

Quetion on Signal Gen Documentation on Wiki #1139

damercer opened this issue Jul 1, 2021 · 3 comments

Comments

@damercer
Copy link

damercer commented Jul 1, 2021

Hi:
I have a question / comment on the documentation for the Signal Generator instrument on the Scopy ADI Wiki page. Don't know if this is the right place to post this vs on EZ.

The comment is:
There does not seem to be any mention of the additive noise feature and how it works. Or am I missing something?

The question is:
Is the added noise waveform calculated once and simply added to the waveform buffer data or is a new random value calculated on the fly sample by sample and added as the waveform buffer is played out?
The first way is least computationally intensive and done on the Host and the second is more computationally intensive and done presumably in the FPGA.

So How is it done and shouldn't that be documented in the Sig Gen Wiki page?

Thanks

Doug

@adisuciu
Copy link
Contributor

adisuciu commented Jul 2, 2021

This is the way it's currently implemented: "the added noise waveform calculated once and simply added to the waveform buffer data"

I'll mention this in the wiki ..

Next step we want to do is actually stream the data which would improve noise randomness (because it is computed on the fly by the computer) but it would only work for low sample rates.

@damercer
Copy link
Author

damercer commented Jul 2, 2021

Great, That's what I figured was happening. That is often sufficient when looking in the time domain but not so much when looking in the frequency domain. The explanation will help users better understand the spectrum of the noise like the one in #1130.

Doug

@adisuciu
Copy link
Contributor

adisuciu commented Jul 5, 2021

I updated the doc. I'll close this if that's ok.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants