ppannuto / python-saleae

Python library to control a Saleae Logic Analyzer
Apache License 2.0
124 stars 55 forks source link

Error running on windows #33

Closed orbitinstasis closed 7 years ago

orbitinstasis commented 7 years ago

Connected devices: <saleae.ConnectedDevice #1 LOGIC_4_DEVICE Logic 4 (4d7e795b740921b3) ACTIVE> Only one Saleae device. Skipping device selection Press Enter to continue...

Setting active channels (digital=[0, 1, 2, 3, 4], analog=[0, 1]) Traceback (most recent call last): File "", line 1, in File "C:\Users\benka\Desktop\python-saleae-master\saleae\saleae.py", line 959, in demo s.set_active_channels(digital, analog) File "C:\Users\benka\Desktop\python-saleae-master\saleae\saleae.py", line 556, in set_active_channels self._finish() File "C:\Users\benka\Desktop\python-saleae-master\saleae\saleae.py", line 167, in _finish ret = self._cmd(', '.join(self._to_send)) File "C:\Users\benka\Desktop\python-saleae-master\saleae\saleae.py", line 200, in _cmd ret = self._recv(expect_nak=expect_nak) File "C:\Users\benka\Desktop\python-saleae-master\saleae\saleae.py", line 191, in _recv raise self.CommandNAKedError saleae.saleae.CommandNAKedError

ppannuto commented 7 years ago

Hey, sorry I didn't see this when it came in.

I don't have a Windows machine, so this is a bit tricky for me to debug.

Actually, I don't think this has anything to do with Windows: image

It looks like the demo tries to set more active channels than the Logic 4 has available, which is why that fails. I can update the demo to fix that

orbitinstasis commented 7 years ago

ahh. I guess that explains it.

Ben

On 13 May 2017 8:48 p.m., "Pat Pannuto" notifications@github.com wrote:

Hey, sorry I didn't see this when it came in.

I don't have a Windows machine, so this is a bit tricky for me to debug.

Actually, I don't think this has anything to do with Windows: [image: image] https://cloud.githubusercontent.com/assets/339422/26028671/7e1f0ecc-37f3-11e7-9aa5-b53b0d33737e.png

It looks like the demo tries to set more active channels than the Logic 4 has available, which is why that fails. I can update the demo to fix that

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/ppannuto/python-saleae/issues/33#issuecomment-301270772, or mute the thread https://github.com/notifications/unsubscribe-auth/AFg6cwns3-5UiSP9lSTAs_v7cuWwOfQ4ks5r5gkPgaJpZM4M5-am .