Skip to content

Tests with hexdigest assume little-endianness #425

Closed

Description

@QuLogic

Minimal, reproducible code sample, a copy-pastable example if possible

$ pytest

Problem description

When running tests on s390x (a big-endian machine), many of them fail checking the hexdigest. This is generally because the data is in a different byte order.

I don't know if tests should be creating little-endian data, or hexdigest should be changed to produce consistent results regardless of byte order.

Version and installation information

Please provide the following:

  • Value of zarr.__version__: 2.3.1
  • Value of numcodecs.__version__: 0.6.3
  • Version of Python interpreter: 3.7.3
  • Operating system (Linux/Windows/Mac): Fedora Rawhide
  • How Zarr was installed (e.g., "using pip into virtual environment", or "using conda"): it's not; I'm building a package.

Activity

  1. QuLogic commented on Apr 2, 2019

    @QuLogic
    ContributorAuthor

    Looking at least at the test_hexdigest ones, they mostly don't contain any data and are just producing a digest of the metadata. In that case, adding specific byte order to the intitialization should be fine and doesn't really affect anything. Also, when they do contain data, it gets converted to the byte order in the metadata, so that works too.

  2. added a commit that references this issue on Apr 2, 2019
    b938b71
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions