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

Make USB communication work again (with my Mac and Oscilloscope, at least) #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

th-in-gs
Copy link

  • (On some platforms?) pyvisa-py uses decimal numbers for the VID/PID values. This change allows both the hex and decimal representations to work when finding the Rigol device.
  • After fixing the above, I could still never get a screenshot to save - there were always errors deep in pyvisa. The problem I was having is documented here: Rigol DS1104z Issues pyvisa/pyvisa#481. '64 - 12' (52) seems to be the maximum chunk size that allows reliable communication. I left it as '64 - 12' in this change rather than simply 52 to match the code quoted in that issue.

Both these changes were required to allow the script to work with my scope (a MSO1075Z), running DS1000Z firmware v00.04.05.02.00, connected to my M2 Mac running macOS Ventura 13.3.1.

- (On some platforms?) pyvisa-py uses decimal numbers for the VID/PID values. This change allows both the hex and decimal representations to work when finding the Rigol device.
- After fixing the above, I could still never get a screenshot to save - there were always errors deep in pyvisa. The problem I was having is documented here: pyvisa/pyvisa#481. '64 - 12' (52) seems to be the maximum chunk size that allows reliable communication. I left it as '64 - 12' in this change rather than simply 52 to match the code quoted in that issue.

Both these changes were required to allow the script to work with my scope (a MSO1075Z), running DS1000Z firmware v00.04.05.02.00, connected to my M2 Mac running macOS Ventura 13.3.1.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant