Open maxim-kukushkin opened 2 years ago
Any chance to get reviews on this PR? Or may be suggestions if community/code owner find that this feature must not be there
any thoughts?
@mkaz, any chance you can have a look at this change?
@mkaz, are you still maintaining the repo or is it abandoned?
Termgraph
is a useful CLI tool and it can become even more useful if it can seamlessly integrate with existing bin utils. One example of the latter one issort
anduniq
commands in Unix-like systems.Adding
| sort | uniq -c
to your pipeline is often useful to get some statistics. The problem is that it prints the number of occurrences first, and then the actual label string. This is goes in contrast to whatTermgraph
expects and thus visualizing these numbers would require an extra pre-processing like anawk
script.This PR adds
--values-first
flag to Termgraph, making it assume that each line first contains the values and labels go as the last column (vs. the default behavior where labels are the first column).ex10.dat
shows that it works with multiple value columns as well. Respective unit test was added.With this change, it's possible to do the following:
| sort | uniq -c | termgraph --values-first
and have it visualize the data.The input data would look like this: