python3Packages.plotly: 6.1.0 -> 6.1.2, unbreak on darwin

[yzx9]

Plotly has been failing on darwin since the bump to 6.1.0.

If I should rebase this branch onto staging, please let me know. I'm also happy to split the version update into a separate PR if that helps get this one merged sooner.

Hydra: https://hydra.nixos.org/job/nixpkgs/trunk/python313Packages.plotly.aarch64-darwin

Things done

• Built on platform(s)
  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
• For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
  • sandbox = relaxed
  • sandbox = true
• Tested, as applicable:
  • NixOS test(s) (look inside nixos/tests)
  • and/or package tests
  • or, for functions and "core" functionality, tests in lib/tests or pkgs/test
  • made sure NixOS tests are linked to the relevant packages
• Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
• Tested basic functionality of all binary files (usually in ./result/bin/)
• Nixpkgs 25.11 Release Notes (or backporting 24.11 and 25.05 Nixpkgs Release notes)
  • (Package updates) Added a release notes entry if the change is major or breaking
• NixOS 25.11 Release Notes (or backporting 24.11 and 25.05 NixOS Release notes)
  • (Module updates) Added a release notes entry if the change is significant
  • (Module addition) Added a release notes entry if adding a new NixOS module
• Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other contributing documentation in corresponding paths.

---

Add a 👍 reaction to pull requests you find important.

[yzx9]

Many unit tests were failing during NumPy import with the following error: RecursionError: maximum recursion depth exceeded. We had previously removed some tests for the same reason, but that patch was dropped in the recent update (#408185). However, the newly added tests are now causing failures on Darwin. Let’s disable them for now.

details

ERROR tests/test_plotly_utils/validators/test_pandas_series_input.py::test_color_validator_categorical[Index] - RecursionError: maximum recursion depth exceeded
= 175 failed, 2855 passed, 21 skipped, 192 deselected, 4 xfailed, 686 warnings, 2 errors in 74.32s (0:01:14) =

error:

tests/test_plotly_utils/validators/test_xarray_input.py:126: 
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
_plotly_utils/basevalidators.py:1395: in validate_coerce
    self.raise_invalid_val(v)
_plotly_utils/basevalidators.py:308: in raise_invalid_val
    v=repr(v),
/nix/store/qzqlix3rhk63zsmf05p1w14cx78rh96c-python3.12-xarray-2025.04.0/lib/python3.12/site-packages/xarray/core/common.py:184: in>
    return formatting.array_repr(self)
/nix/store/q54d9qqlb27g2nf9ja4zsa89gl7b3yvm-python3-3.12.11/lib/python3.12/reprlib.py:21: in wrapper
    result = user_function(self)
/nix/store/qzqlix3rhk63zsmf05p1w14cx78rh96c-python3.12-xarray-2025.04.0/lib/python3.12/site-packages/xarray/core/formatting.py:688>
    data_repr = short_data_repr(arr)
/nix/store/qzqlix3rhk63zsmf05p1w14cx78rh96c-python3.12-xarray-2025.04.0/lib/python3.12/site-packages/xarray/core/formatting.py:657>
    return limit_lines(repr(array.data), limit=40)
/nix/store/4xvvphpbvk5kkiqqnq3hflipbzb7ym2q-python3.12-numpy-2.3.0/lib/python3.12/site-packages/numpy/_core/arrayprint.py:1621: in>
    extras.append(f"dtype={dtype_short_repr(arr.dtype)}")
/nix/store/4xvvphpbvk5kkiqqnq3hflipbzb7ym2q-python3.12-numpy-2.3.0/lib/python3.12/site-packages/numpy/_core/arrayprint.py:1574: in>
    return f"'{str(dtype)}'"
/nix/store/4xvvphpbvk5kkiqqnq3hflipbzb7ym2q-python3.12-numpy-2.3.0/lib/python3.12/site-packages/numpy/__init__.py:852: in <module>
    from . import exceptions
<frozen importlib._bootstrap>:1412: in _handle_fromlist
    ???
/nix/store/4xvvphpbvk5kkiqqnq3hflipbzb7ym2q-python3.12-numpy-2.3.0/lib/python3.12/site-packages/numpy/__init__.py:733: in __getatt>
    import numpy.exceptions as exceptions
/nix/store/4xvvphpbvk5kkiqqnq3hflipbzb7ym2q-python3.12-numpy-2.3.0/lib/python3.12/site-packages/numpy/__init__.py:733: in __getatt>
    import numpy.exceptions as exceptions
E   RecursionError: maximum recursion depth exceeded
!!! Recursion detected (same locals & position)

caused by the following code:

"""
Stand-alone module to provide information about whether optional deps exist.

"""

from importlib import import_module
import logging
import sys

logger = logging.getLogger(__name__)
_not_importable = set()


def get_module(name, should_load=True):
    """
    Return module or None. Absolute import is required.

    :param (str) name: Dot-separated module path. E.g., 'scipy.stats'.
    :raise: (ImportError) Only when exc_msg is defined.
    :return: (module|None) If import succeeds, the module will be returned.

    """
    if not should_load:
        return sys.modules.get(name, None)

    if name not in _not_importable:
        try:
            return import_module(name)
        except ImportError:
            _not_importable.add(name)
        except Exception:
            _not_importable.add(name)
            msg = f"Error importing optional module {name}"
            logger.exception(msg)

    return None

[GaetanLepage]

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 418321 --package python312Packages.plotly --package python313Packages.plotly
Commit: 51a0ea5fb7c3c8eb81e08740870426049cfcf382

---

x86_64-linux

✅ 4 packages built:

• python312Packages.plotly
• python312Packages.plotly.dist (python312Packages.plotly.dist.dist)
• python313Packages.plotly
• python313Packages.plotly.dist (python313Packages.plotly.dist.dist)

---

aarch64-linux

✅ 4 packages built:

• python312Packages.plotly
• python312Packages.plotly.dist (python312Packages.plotly.dist.dist)
• python313Packages.plotly
• python313Packages.plotly.dist (python313Packages.plotly.dist.dist)

---

x86_64-darwin

✅ 4 packages built:

• python312Packages.plotly
• python312Packages.plotly.dist (python312Packages.plotly.dist.dist)
• python313Packages.plotly
• python313Packages.plotly.dist (python313Packages.plotly.dist.dist)

---

aarch64-darwin

✅ 4 packages built:

• python312Packages.plotly
• python312Packages.plotly.dist (python312Packages.plotly.dist.dist)
• python313Packages.plotly
• python313Packages.plotly.dist (python313Packages.plotly.dist.dist)

[Pandapip1]

Diff LGTM