Skip to content

Fix flaky categorical concat test#12276

Merged
crusaderky merged 10 commits intodask:mainfrom
abslock128w:fix-flaky-categorical-concat-test
Feb 5, 2026
Merged

Fix flaky categorical concat test#12276
crusaderky merged 10 commits intodask:mainfrom
abslock128w:fix-flaky-categorical-concat-test

Conversation

@abslock128w
Copy link
Copy Markdown
Contributor

On minimal pandas versions, concatenation of categorical data does not guarantee stable ordering of values, even when categories and dtypes match. The test was asserting positional equality for categorical Series, which is stronger than the guarantees provided by pandas and leads to frequent failures in mindeps-optional.
On minimal pandas versions, concatenation of categorical data does not guarantee stable ordering of values, even when categories and dtypes match. The test was asserting positional equality for categorical Series, which is stronger than the guarantees provided by pandas and leads to frequent failures in mindeps-optional.
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 5, 2026

Unit Test Results

See test report for an extended history of previous test failures. This is useful for diagnosing flaky tests.

     21 files  ±0       21 suites  ±0   5h 11m 31s ⏱️ +51s
 18 282 tests ±0   17 010 ✅ +2   1 272 💤 ±0  0 ❌  - 2 
294 733 runs  ±0  252 411 ✅ +1  42 322 💤 +1  0 ❌  - 2 

Results for commit 8df9027. ± Comparison against base commit 2bb80d7.

♻️ This comment has been updated with latest results.

@crusaderky
Copy link
Copy Markdown
Collaborator

Looks like you have a merge mash in your PR; I see multiple copies of the same code blocks

@abslock128w
Copy link
Copy Markdown
Contributor Author

Looks like you have a merge mash in your PR; I see multiple copies of the same code blocks

i had an internet issue yesterday..
prolly due to which i created more than one commit on the same change will fix it!!

@abslock128w
Copy link
Copy Markdown
Contributor Author

Looks like everything is green now. Thank you for reviewing!!
I’m happy to tweak anything if required.

@crusaderky
Copy link
Copy Markdown
Collaborator

Let's see how this goes long term. Thank you!

@crusaderky crusaderky merged commit 9cc24c1 into dask:main Feb 5, 2026
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Flaky test on mindeps: test_concat_categorical

2 participants