add multi index with categories test #49975
Conversation
|
|
||
|
|
||
| @pytest.mark.parametrize("case", [0.5, "xxx"]) | ||
| @pytest.mark.parametrize( |
There was a problem hiding this comment.
Please avoid changing unrelated lines for style. We use black via precommit to autoformat the code base: https://pandas.pydata.org/pandas-docs/stable/development/contributing_codebase.html#pre-commit
There was a problem hiding this comment.
Sorry about that, ran pre-commit to address
mroeschke
added
the
Testing
|
Failing test seems unrelated to this |
| @pytest.mark.parametrize( | ||
| "b", | ||
| [ | ||
| pd.Categorical(["a", "b"], categories=["b", "a"]), |
There was a problem hiding this comment.
Is one of these supposed to have ordered=True like in the original issue? Otherwise both b params are the same
|
This pull request is stale because it has been open for thirty days with no activity. Please update and respond to this comment if you're still interested in working on this. |
|
Thanks for the pull request, but it appears to have gone stale. If interested in continuing, please merge in the main branch, address any review comments and/or failing tests, and we can reopen. |
|
rebased master and addresses the comment, I dont think the PR sync while it is closed so you cant see the newest commit |
|
For some reason I cannot re-open this pull request. Mind opening up another one? |
2 participants
This PR adds test to address the issue in #49337, which have already been fixed on main
Do you want a entry in the changelog since this is not a fix, just a test to ensure we dont run into this again?
- [ ] Added an entry in the latestdoc/source/whatsnew/vX.X.X.rstfile if fixing a bug or adding a new feature.