ben-e-whitney

Results 21 comments of ben-e-whitney

Here is the [body][body] of `Tree.show`: ```python self._reader = "" def write(line): self._reader += line.decode('utf-8') + "\n" try: self.__print_backend(nid, level, idhidden, filter, key, reverse, line_type, data_property, func=write) except NodeIDAbsentError: print('Tree...

@leonardbinet This solution seems a little strange to me, since 1. the output is sent to `stdout` whether the new parameter is `True` or `False`, 2. the docstring says the...

Would it be better for me to provide an implementation of `contextlib.redirect_stdout` or to avoid its use altogether? I don't know the policy on Python 2 compatibility.

I can reproduce this on 360a2fb853adbb1d758704960d4f51e85e4d3f57. Here is more or less what's happening: 1. The original dataset is transformed into a collection of coefficients. 2. For each coefficient `x`, a...

Apologies for the delay. I have begun working on this and have reproduced the problem. ![image](https://user-images.githubusercontent.com/5350974/165541830-ecfefec0-dd61-4a32-b4ba-6b42349f6ac5.png) I'm guessing there is some issue with the coefficient quantization. Looking into it now....

@lindstro Just an update: I'm working on a bug in the coefficient quantization code. See #190. Once that's done, I'll return to this.

Hi @lindstro. Apologies again for how long this is taking. Fixing the Huffman code ended up entailing a *lot* of work; see #196. Running on that branch, the accuracy gain...

It looks like my compiler is letting me get away with something a little improper. Once the branch is ready to be merged I'll have someone with a Mac try...

@JieyangChen7 This looks like an MGARD-X error, I think? @wangwu1991 Thanks very much for the bug report. Can you tell us how you compiled MGARD and what command you were...

Another issue: since your data is pointwise random, there's very little 'compressible structure' for MGARD to take advantage of. The algorithm can't do much with noise. You should get a...