Skip to content

Conversation

@grandixximo
Copy link

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.

@grandixximo grandixximo mentioned this pull request Jan 20, 2026
@grandixximo
Copy link
Author

I think I pulled at the wrong time, I will try to fix it

Luca Toniolo added 2 commits January 21, 2026 13:10
- 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)
@grandixximo
Copy link
Author

Is retro compatibility important?
Like if switch back to 2.9, you want to open the 2.10 new halscope files, should that be supported?
I would need to add my new settings in a comment as the .halscope file, is that acceptable? or screw retro-compatibility?

2.10 can open 2.9 settings, but 2.9 will not understand 2.10 settings

@grandixximo
Copy link
Author

grandixximo commented Jan 21, 2026

I implemented full bidirectional compatibility, so whomever switches branches won't be bothered by the changes

@hansu
Copy link
Member

hansu commented Jan 21, 2026

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?

@grandixximo
Copy link
Author

grandixximo commented Jan 21, 2026

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...

@andypugh
Copy link
Collaborator

I am not sure that Halscope usefully opens files?
You can save-out the data files but AFAIK you can't load one in to analyse it.
#966
While you are at it.... :-)

@hansu
Copy link
Member

hansu commented Jan 21, 2026

I am not sure that Halscope usefully opens files?

It can open configuration files...

@grandixximo
Copy link
Author

On it

@grandixximo
Copy link
Author

Simple job for my stochastic parrot 🤭
I tested, everything seems in order, you can open a log and view it, even if you don't have the pins of the original logger, phantom log pins will be created for you, and forced to be enabled, the CVS also contains the configuration, so everything will look like what it did in the original log scaling and position, but it can be changed zoomed etc, rolling will clear the loaded log.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants