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

Find a better way for controlling groups of 7S-display digits (e.g. W1401, W1219) #20

Closed
TG9541 opened this issue Apr 13, 2017 · 4 comments
Assignees

Comments

@TG9541
Copy link
Owner

TG9541 commented Apr 13, 2017

Blank, tab, or CR might be suitable.

@TG9541
Copy link
Owner Author

TG9541 commented Apr 14, 2017

Solution details:

  • "blank" moves to the next 7S-LED group, and erases it, wrap around to first group
  • CR (actually LF) moves to the last group
    This is a bit hacky but it works OK.

On a W1219, the following code writes "F.o.o" to the 1st (red) 7S-LED group, and the timer value to the 2nd (green) group.

FILE
: p cr ."  F.o.o" tim . ; 
' p bg !
HAND

@TG9541
Copy link
Owner Author

TG9541 commented Apr 15, 2017

The first prototype code works but for transparency and ease of use the following requirements shall apply:

  • CR shall move the output to the first "tab" group without changing the display contents
  • whitespace shall move the output to the next 7S-LED group only if it was not preceded another move
  • whitespace shall clear the current 7S-LED group, whether the output has been moved, or not

TG9541 added a commit that referenced this issue Apr 16, 2017
@TG9541
Copy link
Owner Author

TG9541 commented Apr 16, 2017

The feature now works as described in the previous comment.

@TG9541 TG9541 closed this as completed Apr 16, 2017
@TG9541 TG9541 reopened this Apr 17, 2017
@TG9541
Copy link
Owner Author

TG9541 commented Apr 17, 2017

A review revealed that DCDC and W1209 had a problem with the new semantics of HAS_LED7SEG which no longer (kind of) encodes the number of 7S-LED digits but that of the number of LED groups.

TG9541 added a commit that referenced this issue Apr 17, 2017
@TG9541 TG9541 closed this as completed Apr 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant