Skip to content

MAINT: Fix CircleCI#13351

Merged
drammock merged 2 commits into
mne-tools:mainfrom
larsoner:circle
Jul 29, 2025
Merged

MAINT: Fix CircleCI#13351
drammock merged 2 commits into
mne-tools:mainfrom
larsoner:circle

Conversation

@larsoner
Copy link
Copy Markdown
Member

@larsoner larsoner commented Jul 25, 2025

Will fail until pyvista/pyvista#7746 lands. In the meantime, we should be able to see the improved traceback in the CircleCI output of the touched example (which is failing on main) that takes

    3 vtkPolyData @ mne/conf.py:Resetter.__call__:after:70_fnirs_processing.py:
    list: len=1, 4 referrers: dict/list
    list: len=1, 4 referrers: dict/list
    list: len=1, 4 referrers: dict/list

and makes it a bit more informative:

    3 vtkPolyData @ mne/conf.py:Resetter.__call__:after:70_fnirs_processing.py:
    list[0] with len=1 | 3 referrers: dict['_glyph_geom'] / list[1] / list[743437]
    list[0] with len=1 | 3 referrers: dict['_glyph_geom'] / list[1] / list[743441]
    list[0] with len=1 | 3 referrers: dict['_glyph_geom'] / list[1] / list[743444]

which made it way faster to diagnose than before (I could git grep _glyph_geom and see it had to be a PyVista regression rather than an MNE one).

@drammock drammock enabled auto-merge (squash) July 29, 2025 15:55
@drammock drammock merged commit 1c9ac7c into mne-tools:main Jul 29, 2025
29 of 32 checks passed
@drammock drammock deleted the circle branch July 29, 2025 16:12
WouterKroot pushed a commit to WouterKroot/mne-python that referenced this pull request Aug 13, 2025
zEdS15B3GCwq pushed a commit to zEdS15B3GCwq/mne-python that referenced this pull request Aug 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants