Skip to content

armv7 CI failures with recent serialization changes #19632

Closed

Description

@astrofrog

Description

There are currently CI failures on armv7 following (I think) #18703 - @taldcroft would you be able to take a look? (if you open a PR to fix this, add the Extra CI label - thanks!)

______________ [doctest] astropy.table.connect.represent_indices _______________
  288 
  289 The format is most easily illustrated via the ECSV output in this example::
  290 
  291   >>> from astropy.table import QTable
  292   >>> import sys
  293   >>> t = QTable()
  294   >>> t["a"] = [2, 3, 1]
  295   >>> t["b"] = [3, 5, 4]
  296   >>> t.add_index("a")
  297   >>> t.write(sys.stdout, format="ecsv", write_indices=True)
  Differences (unified diff with -expected +actual):
      @@ -3,11 +3,11 @@
       # datatype:
       # - name: a
      -#   datatype: int64
      +#   datatype: int32
       #   meta: !!omap
       #   - __indices__:
       #     - colnames: [a]
       #       index_colname: __index__
      -# - {name: b, datatype: int64}
      -# - {name: __index__, datatype: int64}
      +# - {name: b, datatype: int32}
      +# - {name: __index__, datatype: int32}
          '#       index_colname: __index__',
    -     '# - {name: __index__, datatype: int64}',
    ?                                         ^^
    +     '# - {name: __index__, datatype: int32}',
    ?                                         ^^
          '# schema: astropy-2.0',
          'a __index__',
          '1 0',
          '3 2',
          '2 1',
      ]
  FAILED astropy/table/tests/test_index.py::test_indices_serialization_representation_multiple - AssertionError: assert ['# %ECSV 1.0... !!omap', ...] == ['# %ECSV 1.0... !!omap', ...]
    
    At index 4 diff: '#   datatype: int32' != '#   datatype: int64'
    
    Full diff:
      [
          '# %ECSV 1.0',
          '# ---',
          '# datatype:',
          '# - name: a',
    -     '#   datatype: int64',
    ?                       ^^
    +     '#   datatype: int32',
    ?                       ^^
          '#   meta: !!omap',
          '#   - __indices__:',
          '#     - colnames: [a, __index__1]',
          '#       index_colname: __index__',
          '#       primary: true',
          '#     - colnames: [a]',
          '#       index_colname: __index__2',
    -     '# - {name: __index__1, datatype: int64}',
    ?                                          ^^
    +     '# - {name: __index__1, datatype: int32}',
    ?                                          ^^
    -     '# - {name: __index__, datatype: int64}',
    ?                                         ^^
    +     '# - {name: __index__, datatype: int32}',
    ?                                         ^^
    -     '# - {name: __index__2, datatype: int64}',
    ?                                          ^^
    +     '# - {name: __index__2, datatype: int32}',
    ?                                          ^^
          '# schema: astropy-2.0',
          'a __index__1 __index__ __index__2',
          '1 5 0 0',
          '3 4 2 2',
          '2 3 1 1',
      ]
  = 5 failed, 26782 passed, 3899 skipped, 72 deselected, 214 xfailed in 2455.83s (0:40:55) =
  Error: The process '/home/runner/work/_actions/uraimo/run-on-arch-action/d94c13912ea685de38fccc1109385b83fd79427d/src/run-on-arch.sh' failed with exit code 1

Expected behavior

No response

How to Reproduce

  1. Get package from '...'
  2. Then run '...'
  3. An error occurs.
# Put your Python code snippet here.

Versions

import astropy
astropy.system_info()
# Paste the result here

Activity

  1. added this to the v8.0.0 milestone on May 1, 2026
  2. neutrinoceros commented on May 1, 2026

    @neutrinoceros
    Contributor

    This seems trivial to fix. I'll open a PR shortly

  3. neutrinoceros commented on May 1, 2026

    @neutrinoceros
    Contributor

    It was not, in fact, trivial. Hopefully it's still easy to fix, just not as easy as I thought. Will inquire.

  4. pllim commented on May 1, 2026

    @pllim
    Member
  5. taldcroft commented on May 2, 2026

    @taldcroft
    Member

    See #19644. Hopefully this passes on ARMv7.

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

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions