tenox7 / ttyplot

a realtime plotting utility for terminal/console with data input from stdin
Apache License 2.0
961 stars 43 forks source link

option -e and -s/-m #40

Closed servidge closed 4 years ago

servidge commented 5 years ago

It seems the options does not work properly. please correct me if i am wrong.

The tested version is compiled from today's source. 1.4

PS: despite that a cool piece of software ;)

tenox7 commented 4 years ago

You are absolutely right about -e not being honored right now. It's a bug. It was missing in optstring. Fixed now.

You are also right about -s not being used when -m is specified. However this is by design. Think of flag -s is not a limit of any kind. It's an initial or minimal graph or scale height. Most commonly used when for measuring something that has a known scale like CPU usage in % (the top being 100%) or link bandwidth (the top being xxx MB/s). You use -s when you don't want an actual limit but you want graph to be scaled minimally at x value. You use -m when you want a completely fixed scale never to go lover or higher than the value of the flag. One can think that -m is inclusive of -s and therefore specifying -s has no effect. Let me know if I can word it better way in help/description/usage.

tenox7 commented 4 years ago

I have updated the usage regarding -s and -m to be less confusing:

  -s minimum/initial scale of the plot (can go above if data input has larger value)
  -m maximum value, if exceeded draws error line (see -e), plot scale is fixed
tenox7 commented 4 years ago

fixed in head:master

servidge commented 4 years ago

Ok, i can confirm the fix. And the -s or -m parameters are used either or and not both at the same time. If you use the -s the -e does not take effect. i can live with that. thanks for the fix

tenox7 commented 4 years ago

if -m is used -s becomes redundant, also if -s (and not -m) then -e is never used, so... yes :)