James Antill comments

Results 19 comments of


                                            James Antill

Choose and filter don't work well with long lines.

Both here. left/right arrows move to the next "page" for long results or move to the top/bottom for small results. Dito. h/l keys.

Choose and filter don't work well with long lines.

To be clear, this is a left/right problem ... not an up/down problem. Eg. 200 character lines on an 80 character wide terminal.

Option to use negative style for inline change highlights

FWIW I ended up here because the default highlighting is unreadable for my poor old eyes (looks smudged). I ended up with: [color "diff-highlight"] oldHighlight = reverse red oldReset =...

Performance is non-linear, thus. very bad as input gets large

Note that this isn't just you, "github.com/btcsuite/btcutil/base58" gives: % time ./base58-2 -i abcd10 | wc -c 13658 ./base58-2 -i abcd10 0.27s user 0.01s system 96% cpu 0.288 total % time...

Performance is non-linear, thus. very bad as input gets large

Yeh, it's probably worth pointing that out somewhere obvious. I ran across it when I heard about multihash/ipfs and the information about it mostly suggested it as a better replacement...

Performance is non-linear, thus. very bad as input gets large

I ended up writing a base50 API, which had the features I wanted from base58 ... probably overkill. Feel free to close.

Two time series with both Y axes using identical min-max range

Yes, if I understand you correctly, look at the twoaxis example code ... and pass a Range{Min: x, Max: y} to both y axis (with the same values).

Fix human readable output edge cases

I added testing, and realized you used KiB format not KB ... so I changed it over to base 10 and created formatBytesKiB(), as you can see that produces output...

Choose and filter don't work well with long lines.

I don't care, as long as I can see the data somehow. The obvious two are line wrapping and scrolling the viewport, with the arrows, to the right. On Wed,...

UID lookup is bad

You could also save a bit of space/time by creating the map as map[uint64]string and then testing before you have to do strconv.Itoa().