<_whitenotifier-5>
[scopehal] azonenberg edited issue #237: LeCroy: add high speed serial trigger support (8B10B/NRZ/64b66b) - https://git.io/JUBKq
<_whitenotifier-5>
[scopehal] azonenberg commented on issue #237: LeCroy: add high speed serial trigger support (8B10B/NRZ/64b66b) - https://git.io/JYK1O
<_whitenotifier-5>
[scopehal] azonenberg opened issue #400: LeCroy: Support for SDA 8Zi memory depth/sample rate tables - https://git.io/JYKDT
<_whitenotifier-5>
[scopehal] azonenberg assigned issue #400: LeCroy: Support for SDA 8Zi memory depth/sample rate tables - https://git.io/JYKDT
<_whitenotifier-5>
[scopehal] azonenberg labeled issue #400: LeCroy: Support for SDA 8Zi memory depth/sample rate tables - https://git.io/JYKDT
<azonenberg>
aaand apparently the 10baseT decoder isnt happy with me
<azonenberg>
seems there's a regression, my saved example waveform doesnt decode either
<monochroma>
D:
<azonenberg>
(the long term plan is to develop a unit test suite based on some of these saved waveforms)
<azonenberg>
aha
<azonenberg>
it assumed 10 Mbps and hard coded ui width in timebase units
<azonenberg>
never got updated when we changed from ps to fs
<_whitenotifier-5>
[scopehal] azonenberg pushed 1 commit to master [+0/-0/±1] https://git.io/JYKQN
<Degi>
Somehow I managed to make a power supply from ordinary components which makes EMI with rise time beyond my scopes bandwidth
<azonenberg>
lol
m4ssi has quit [Remote host closed the connection]
<Degi>
(I think its the UF4007 diodes getting a bit of reverse current in the 75 ns it takes to turn off and having the step recovery diode effect)
<_whitenotifier-5>
[scopehal] mubes forked the repository - https://git.io/JYiFA
<_whitenotifier-5>
[scopehal] mubes opened pull request #402: Changes to support SDS2000+ scope - https://git.io/JYiAM
<d1b2>
<mubes> Right, assuming I didn't royally screw that up you should have some alpha level commits for Siglent SDS2000 support. Comments and brickbats welcome. There's quite a lot more can be done but it's now better than it was and if someone starts using it then it will improve.
<azonenberg>
xptools fix merged
<azonenberg>
mubes: in your scopehal PR i see a bunch of spaces instead of \t's for indentation
<azonenberg>
convention is one \t = one level of indentation = four columns
<d1b2>
<mubes> It's gone through clang-format so it should be correct....unless I managed to break it somehow. Which files, all of them?
<azonenberg>
yeah looks like it unless github is misrendering the content
<d1b2>
<mubes> No, it'll be me
<d1b2>
<mubes> I'll update the PR tomorrow
<azonenberg>
also interesting there is a 50 ohm AC coupled option on siglent?
<azonenberg>
i've never seen that anywhere else
<azonenberg>
You can put that in the API but there is no way in the GUI to use it yet
<azonenberg>
I actually think i need to add a GetSupportedCoupling() or similar API
<azonenberg>
to the Oscilloscope class
<d1b2>
<mubes> Well, I can set it...no idea if its usable
<azonenberg>
so that it reports the set of couplings that different instruments supports
<azonenberg>
e.g. a lot of lower end stuff has no 50 ohm option
<azonenberg>
so that shouldnt even show up in the gui
<d1b2>
<mubes> (Is there a way to reset the vertical offset or time offset to zero quickly?
<d1b2>
<mubes> Hmm...looks like the original file is correctly formatted;