labrad / servers

LabRAD servers
24 stars 21 forks source link

Deconvolution - Settling and Reflection rates and amplitudes seem incorrectly applied. #383

Closed btchiaro closed 7 years ago

btchiaro commented 7 years ago

It appears as though settling amplitudes are being applied with the reflection rates. It's possible that this output is correct and we don't fully understand the physics. If so please advise. Our data is below. Running spectroscopy_z_func on master pyle, master servers, we observe the following pulses, where the magenta trace is the pulse to which the corrections should be applied:

Our control experiment has all amplitudes set to zero, so that there should be no compensation for settling times or reflections. With parameters: settlingRates:[0.1 GHz] settlingAmplitudes:[0] reflectionRates:[0.001 GHz] reflectionAmplitudes:[0] We observe: wp_20161025_15_13_29_pro

Now we set a large reflection amplitude and do not observe a correction to the pulse. With parameters: settlingRates:[0.1 GHz] settlingAmplitudes:[0] reflectionRates:[0.001 GHz] reflectionAmplitudes:[0.5] We observe: wp_20161025_15_15_36_pro

If we turn on the settling amplitude, with the reflection amplitude set to zero, we observe what appears to be a reflection correction at a time that seems related to the reflection rate. With:
settlingRates:[0.1 GHz] settlingAmplitudes:[0.5] reflectionRates:[0.001 GHz] reflectionAmplitudes:[0.0] We observe: wp_20161025_15_16_39_pro

We check this by halving the settling amplitude and see that the correction pulse is halved. With: settlingRates:[0.1 GHz] settlingAmplitudes:[0.25] reflectionRates:[0.001 GHz] reflectionAmplitudes:[0.0] We observe: wp_20161025_15_18_43_pro

By changing the reflection rate we are able to change the timing of the correction pulse. With: settlingRates:[0.1 GHz] settlingAmplitudes:[0.25] reflectionRates:[0.002 GHz] reflectionAmplitudes:[0.0] We observe: wp_20161025_15_19_54_pro

@zchen088 @maffoo @JulianSKelly @DanielSank Thanks!

zchen088 commented 7 years ago

Ah yes, I found a typo.

zchen088 commented 7 years ago

Note that this server actually lives in pyle, and I've fixed it there.

btchiaro commented 7 years ago

Thanks for the fix Jimmy! I'm pulling this now.

On Tue, Oct 25, 2016 at 4:04 PM, zchen088 notifications@github.com wrote:

Note that this server actually lives in pyle, and I've fixed it there.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/martinisgroup/servers/issues/383#issuecomment-256203964, or mute the thread https://github.com/notifications/unsubscribe-auth/AIeWl6yBajCw7WrmrM2K1mWGTA2SXI5Pks5q3orjgaJpZM4Kgj5D .