Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Trailing whitespaces #8

Open
danilobellini opened this issue Oct 19, 2016 · 0 comments
Open

Trailing whitespaces #8

danilobellini opened this issue Oct 19, 2016 · 0 comments

Comments

@danilobellini
Copy link
Collaborator

As of today, hipsterplot "prints" its plots with trailing whitespaces. I think it shouldn't, and my use case is the use of hipsterplot as doctests.

I'm using hipsterplot on PyScanPrev to help me use the examples as a test suite. Today, the Single Pole Lowpass IIR filter and the State-space model examples require hipsterplot, and that gave me some headache due to the trailing whitespaces (as I configure my editor to trim them), which I fixed on this commit by patching hipsterplot to use an alternative printer.

It can be as easy as that patch, hipsterplot can have a print("<some string stuff>".rstrip()) or use an alternative print function. But there are other approaches, e.g. using (optional) delimiters on the plots, or even making the trailing whitespaces something optional using a flag as a keyword argument. What do you think?

dmyersturnbull added a commit to dmyersturnbull/hipsterplot that referenced this issue May 25, 2020
dmyersturnbull added a commit to dmyersturnbull/hipsterplot that referenced this issue May 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant