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

Add LCDPROC NTP Screen #1277

Closed
wants to merge 0 commits into from
Closed

Add LCDPROC NTP Screen #1277

wants to merge 0 commits into from

Conversation

elvisimprsntr
Copy link

@elvisimprsntr elvisimprsntr commented Jul 31, 2023

Requested updates from @jim-p

Original work by @georou

https://redmine.pfsense.org/issues/14625

Copy link
Contributor

@jim-p jim-p left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Testing this out a couple things seem less than ideal:

  1. The screen has no label so it's not obvious what it is. At least printing "NTP: " before the IP address would be helpful
  2. The "(stratum n)" wastes a ton of space on something I'm not sure is significant and it obscures the IP address more often than not. "Stratum" could be abbreviated to "St" maybe or omitted entirely. It would be even worse for an IPv6 address
  3. There is quite a bit of dead space after the Delay/Offset/Jitter labels, though I'm not sure anything can be done for that. The numbers might be able to fit on a single line going horizontally on two lines instead of three. Like we do for the system stats or UPS info. That would free up a whole line to print the Title/IP address and stratum info separately.

@elvisimprsntr
Copy link
Author

elvisimprsntr commented Aug 1, 2023

@jim-p @georou

1,2. Displaying the sync source on a single line is consistent with the web interface. There will never be enough characters to display the reference, so scrolling the row might be the only option.

Screenshot 2023-07-31 at 10 18 00 PM
  1. Problem is the delay, offset, and jitter are in msec. So if someone is not using a GPS or PPS, the values can be very large. We would have to truncate the number of characters.

Perhaps something like this?

LCD Display Generator: http://avtanski.net/projects/lcd/

Screenshot 2023-08-01 at 6 47 58 AM

Or one of the following

Screenshot 2023-08-01 at 6 54 07 AM Screenshot 2023-08-01 at 9 22 06 AM Screenshot 2023-08-01 at 9 45 10 AM

I'm leaning towards the last one. Let me know what you both think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants