question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Thousands separator for progress stats

See original GitHub issue

Right now, progress stat line (EC.TLC_PROGRESS_STATS) looks like this:

Progress(32) at 2019-02-19 20:28:11: 516608828 states generated (432,270 s/min), 19621926 distinct states found (14,277 ds/min), 1433419 states left on queue.

There are some huge numbers that need thousands separators for readability. I propose it to look like this:

Progress(32) at 2019-02-19 20:28:11: 516,608,828 states generated (432,270 s/min), 19,621,926 distinct states found (14,277 ds/min), 1,433,419 states left on queue.

I’m happy to implement this change, but I wanted to first check with you whether you see any issues with this enhancement.

Issue Analytics

  • State:closed
  • Created 5 years ago
  • Comments:7 (7 by maintainers)

github_iconTop GitHub Comments

1reaction
lemmycommented, Feb 19, 2019

@philipmw A patch will be accepted if a) it uses the system’s locale to select the thousands delimiter b) the Toolbox’s parser - which reads TLC’s output - does not break or gets changed correspondingly

To work around b) it might be acceptable to only print a thousands delimiter in TLC’s non-Tool mode

0reactions
lemmycommented, Apr 16, 2019

@philipmw Please reopen if you ever have time to work on the idea outlined in the second paragraph of my previous comment.

Read more comments on GitHub >

github_iconTop Results From Across the Web

Thousands Separator (-numsep) - Progress Documentation
Use Thousands Separator ( -numsep ) to change the character that represents a number's thousands separator in formatted text. The default thousands separator...
Read more >
Thousands separator commas in processing progress row count ...
I know this is the most nitpicky thing ever, but i can't tell wtf number this is lol! :) image. The text was...
Read more >
Should the network use thousands separators in numbers?
Currently the network has a very inconsistent track record when it comes to injecting thousands separators in numbers >= 4 digits.
Read more >
Show or hide the thousands separator - Microsoft Support
To display or hide the thousands separator, select or clear the Use 1000 Separator (,) check box. Tip: To quickly display the thousands...
Read more >
Thousands separator in tagset Excelxp - SAS Communities
For TABULATE purposes, the VAR statement STYLE= override only impacts the variable HEADER cells. To impact the calculated statistics for one ...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found