Skip to content

Conversation

@n-poulsen
Copy link
Contributor

Tracklet stitcher identity tests fail with scipy>=1.11.0. Both test_tracklet and test_stitcher_with_identity fail as the created tracklets have the wrong identities.

@n-poulsen n-poulsen added the dependencies Pull requests that update a dependency file label Jun 27, 2023
@n-poulsen n-poulsen requested review from MMathisLab and jeylau June 27, 2023 07:08
@MMathisLab MMathisLab merged commit 8bbbd1c into main Jun 27, 2023
@MMathisLab MMathisLab deleted the pin_scipy branch July 28, 2023 12:04
hmaarrfk added a commit to hmaarrfk/DeepLabCut that referenced this pull request Sep 25, 2023
See warning in scipy 1.9's documentation.
See warning in https://docs.scipy.org/doc/scipy-1.9.0/reference/generated/scipy.stats.mode.html

The SPEC0 suggests that scipy 1.8 be dropped in 2024, but I think it is
OK to drop it a little early.

If you don't want to drop it, we could add a version check for the
keyword argument, but that seems a little much.

The keyword argument `keepdims` was added in scipy 1.9

xref: DeepLabCut#2290
MMathisLab pushed a commit that referenced this pull request Oct 3, 2023
See warning in scipy 1.9's documentation.
See warning in https://docs.scipy.org/doc/scipy-1.9.0/reference/generated/scipy.stats.mode.html

The SPEC0 suggests that scipy 1.8 be dropped in 2024, but I think it is
OK to drop it a little early.

If you don't want to drop it, we could add a version check for the
keyword argument, but that seems a little much.

The keyword argument `keepdims` was added in scipy 1.9

xref: #2290
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants