You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Also suggest removing the compression information (nbytes_stored, number of chunks initialized) as these can take some time to retrieve. Suggest moving these to a method or property, e.g., storage_info or something like that.
I suppose that would be included too. So using an OrderedDict would help.
Though was more referring to details like the compression information that seem to vary slightly even on the same data. Things like name, shape, dtype, and chunk shape should be fine.
Yes the compression information has been a bit of a bane, small changes can
make the doctests on docstring examples and the tutorial fail, which is a
pain to keep up to date. In the numcodecs PR I ended up putting a load of
ellipses in to try and reduce maintenance overhead.
It would be nice to simplify the repr. It has been useful for me to see
compression info easily to hand, but maybe we could think of an alternate
way to access that information that is almost as convenient as repr.
Activity
alimanfoo commentedon Jan 16, 2017
Also suggest removing the compression information (nbytes_stored, number of chunks initialized) as these can take some time to retrieve. Suggest moving these to a method or property, e.g., storage_info or something like that.
jakirkham commentedon Mar 6, 2017
Yeah, it would also be nice if
reprreturned something that was relatively static between runs.alimanfoo commentedon Mar 6, 2017
jakirkham commentedon Mar 7, 2017
I suppose that would be included too. So using an
OrderedDictwould help.Though was more referring to details like the compression information that seem to vary slightly even on the same data. Things like name, shape, dtype, and chunk shape should be fine.
alimanfoo commentedon Mar 7, 2017
jakirkham commentedon Mar 7, 2017
Was also thinking of doctests when thinking about repr. Totally agree with having a different method provide this.
As a side note, the dictionary changes in Python 3.6 are well explained in this message.