Skip to content

Fix randomly failing multi-animal test script#2233

Merged
AlexEMG merged 1 commit intomainfrom
fix_multi_testscript
May 5, 2023
Merged

Fix randomly failing multi-animal test script#2233
AlexEMG merged 1 commit intomainfrom
fix_multi_testscript

Conversation

@jeylau
Copy link
Contributor

@jeylau jeylau commented May 5, 2023

As data are randomly created in the test script, the TrackletStitcher will occasionally fail to reconstruct 3 tracks. The ValueError is now caught, and a fake data file is written to disk to prevent the test from prematurely exiting.

(Note that the TrackletStitcher is thoroughly tested in https://github.com/DeepLabCut/DeepLabCut/blob/main/tests/test_stitcher.py)

@jeylau jeylau requested a review from AlexEMG May 5, 2023 08:36
@jeylau jeylau self-assigned this May 5, 2023
@jeylau jeylau added the enhancement New feature or request label May 5, 2023
@AlexEMG AlexEMG merged commit 6046238 into main May 5, 2023
@jeylau jeylau deleted the fix_multi_testscript branch May 5, 2023 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants