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
Sayma SAWG produces distorted output at certain amplitudes #1039
Comments
Might be a duplicate of #1022 indeed - I am not sure. |
Also not fixed by JESD update to m-labs/jesd204b@25fd79d |
Reproduced: Artiq: c28fe47, Misoc: 86c6fcb, Migen: 6425844, jesd204b: 25fd79d |
Thanks. |
@marmeladapk Thanks for reproducing! @jordens Do you think this could be SAWG (vs JESD)? |
Does this work for all sawgs? e.g. can I use |
@hartytp any channel |
Flashed this to the startup kernel: from artiq.experiment import *
class SAWGTest(EnvExperiment):
def build(self):
self.setattr_device("core")
self.setattr_device("sawg0")
@kernel
def run(self):
self.core.reset()
self.sawg0.reset()
delay(300 * us)
self.sawg0.frequency0.set(10*MHz)
self.sawg0.amplitude1.set(0.2) Hooked up microscope probes to Sayma https://github.com/hartytp/artiq/tree/sawg_probes
Edit: dac output looks very distorted. |
amplitude to 0.5 gives a good sine and
|
@jordens Is this still open? cf https://irclog.whitequark.org/m-labs/2018-06-09#22299432; Closed status following commit implies to me that I can reproduce fix by rebuilding. |
@jbqubit Please try it, the Sayma in HK is currently broken. https://irclog.whitequark.org/m-labs/2018-06-09#1528542903-1528545687; |
This bug seems fixed now. See the following video: https://drive.google.com/file/d/1_Wkg8eDb3q3fVKrkaSxhvNfwyYxdB2Y1/view?usp=sharing Code: from artiq.experiment import *
import numpy as np
class SAWGTest(EnvExperiment):
def build(self):
self.setattr_device("core")
self.setattr_device("sawg0")
self.amps = np.linspace(0., 1., num=1000)
@kernel
def run(self):
self.core.reset()
self.sawg0.reset()
delay(300 * us)
while True:
for amp in self.amps:
self.sawg0.frequency0.set(10*MHz)
self.sawg0.amplitude1.set(amp)
delay(10*ms) |
Looks relatively normal:
Then change the amplitude to 0.2:
The text was updated successfully, but these errors were encountered: