jakirkham / zarr / 165
100%

DEFAULT BRANCH: master
Ran
Jobs 4
Files 29
Run time 4322min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
165

push

travis-ci

web-flow 
Bump Numcodecs requirement to 0.6.2 (#352)

* Bump Numcodecs requirement to 0.6.1

* Assert MsgPack round-trips bytes objects correctly

Previously MsgPack was turning bytes objects to unicode objects when
round-tripping them. However this has been fixed in the latest version
of Numcodecs. So correct this test now that MsgPack is working
correctly.

* properly guard against removal of object codec

* Ensure `chunk` in `_decode_chunk` is an `ndarray`

* Reshape `chunk` ourselves since it is an `ndarray`

As we already ensured the `chunk` is an `ndarray` viewing the original
data, there is no need for us to do that here as well. Plus the checks
performed by `ensure_contiguous_ndarray` are not needed for our use case
here. Particularly as we have already handled the unusual type cases
above. We also don't need to constrain the buffer size. As such the only
thing we really need is to flatten the array and make it contiguous,
which is what we handle here directly.

* Refactor `reshape` from `_decode_chunk`

As both the expected `object` case and the non-`object` case perform a
`reshape` to flatten the data, go ahead and refactor that out of both
cases and handle it generally. Simplifies the code a bit.

* Consolidate type checks in `_decode_chunk`

As refactoring of the `reshape` step has effectively dropped the
expected `object` type case, the checks for different types is a little
more complicated than needed. To fix this, basically invert and swap the
case ordering. This way we can handle all generally expected types first
and simply cast them. Then we can raise if an `object` type shows up and
is unexpected.

* Drop `ensure_bytes` definition from `zarr.storage`

As Numcodecs now includes a very versatile and effective `ensure_bytes`
function, there is no need to define our own in `zarr.storage` as well.
So go ahead and drop it.

* Take flattened array views to avoid some copies

Make use of Numcodecs' `ensu... (continued)

8338 of 8338 relevant lines covered (100.0%)

3.96 hits per line

Jobs
ID Job ID Ran Files Coverage
1 165.1 0
98.85
 Travis Job 165.1
2 165.2 0
98.82
 Travis Job 165.2
3 165.3 0
98.82
 Travis Job 165.3
4 165.4 0
99.4
 Travis Job 165.4
Source Files on build 165
Detailed source file information is not available for this build.