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.

Sign up for GitHub

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

Jump to bottom

Make target comms confirmation optional. #13

Open
ben-marshall opened this issue Nov 19, 2019 · 0 comments
Open

Make target comms confirmation optional. #13

ben-marshall opened this issue Nov 19, 2019 · 0 comments
Assignees
@ben-marshall
Labels
enhancement New feature or request host issue affects code running on host computer. target Issue affects code running on the target device

Comments

@ben-marshall
Copy link
Contributor

  • Currently every command sent to the target device from the host generates a blocking response code.
  • Removing this makes communication 3x faster.
  • Add ability to turn it on/off.
@ben-marshall ben-marshall added enhancement New feature or request target Issue affects code running on the target device host issue affects code running on host computer. labels Nov 19, 2019
@ben-marshall ben-marshall self-assigned this Nov 19, 2019
ben-marshall added a commit that referenced this issue Nov 19, 2019
@ben-marshall
Turn off command response codes in target comms.
ba1391d 
- Makes communication *much* faster!
- See #13.
- Need to make it configurable.

 On branch dev/v2
 Changes to be committed:
	modified:   scass/comms/Target.py
	modified:   scass/scope/Picoscope5000.py
	modified:   scass/scope/Scope.py
	modified:   scass/ttest/TTestCapture.py
	modified:   target/scass/scass_target.c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ben-marshall ben-marshall

Labels
enhancement New feature or request host issue affects code running on host computer. target Issue affects code running on the target device
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ben-marshall