clifford icon indicating copy to clipboard operation
clifford copied to clipboard

Should ` _settings._eps` be used in printing?

Open tBuLi opened this issue 3 years ago • 1 comments

In the __str__ method of MultiVector, there is the following line:

if abs(coeff) < _settings._eps:
    continue  # too small to print

But this doesn't feel right to me since e.g. I'm trying to compute inverses so I need _eps to be really tiny, but that shouldn't impact printing. I have the feeling _settings._print_precision should be used here just like everywhere else in this method but since it isn't, perhaps there is a reason I'm missing?

If it is a bug let me know, I can make a quick PR.

tBuLi avatar Oct 14 '20 08:10 tBuLi

See also #356

eric-wieser avatar Oct 14 '20 12:10 eric-wieser