-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Halscope always 16 channels, and add user configurable samples #3722
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
base: master
Are you sure you want to change the base?
Conversation
|
I think I pulled at the wrong time, I will try to fix it |
- Add GUI control for sample count in acquire dialog with config file persistence (SAMPLES command read before scope_rt loads) - Always enable all 16 channels - remove confusing record length radio buttons that forced tradeoff between channels and samples - Initialize sample_len and rec_len in init_horiz() so sampling works immediately without opening acquire dialog - Add bounds checking for num_samples (min 1000, max 1000000) - Increase default window size to 1050x550 for better channel display - Show restart message when sample count changed (requires reload)
|
Is retro compatibility important? 2.10 can open 2.9 settings, but 2.9 will not understand 2.10 settings |
|
I implemented full bidirectional compatibility, so whomever switches branches won't be bothered by the changes |
|
What happens if 2.9 halscope reads a 2.10 halscope file? If it still reads all "old" parameters except the new ones it would be no problem. But I haven't looked into it if it may fail to read the file at all? |
Previous commit 2.9 would just not open the new files, probably cause I had the new config in the header, but I think it makes more sense to have it in the header as a comment which 2.10 can parse, and 2.9 ignores, already pushed the commit to fix this... |
|
I am not sure that Halscope usefully opens files? |
It can open configuration files... |
|
On it |
|
Simple job for my stochastic parrot 🤭 |
The default memory has been increased, and user can set custom memory and restart to get less or more.
During the work on the jerk control have been using the scope a lot, this makes it a better tool for debugging.