search

annacrombie / plot

plot on the command line
MIT License
113 stars 9 forks source link

feat: add fixed plot bounds option #11

Closed AdrianDC closed 3 years ago

AdrianDC commented 3 years ago

@holmanb

Refactored a bit, and isolated changes to commits. Adding this feature to the tests of #10.

AdrianDC commented 3 years ago

Removed the limitations from the original commit as the bounds are not related to the rendering height / width, but to the values themselves, thus do not require any limit.

Test: seq 1 1000 | ./build/plot a 10 -b -10:1000 -c Y -f -S 200 -x 5: -y 10:3:1

holmanb commented 3 years ago

@AdrianDC - This looks good to me. It's approximately what I had going at this commit, with some improvements.

AdrianDC commented 3 years ago

Yes @holmanb , ported from your commit + cleanups & documentation mentioned above, hence the original authorship + original commit date. :+1:

holmanb commented 3 years ago

@AdrianDC - Ah, I see that now. I didn't see the commit history before.

AdrianDC commented 3 years ago

The next patchset will also fix double typed bounds to allow -p 0.123:0.234

AdrianDC commented 3 years ago

Follows #5