Skip to content

Fix mypy errors caused by new version of mypy#5485

Merged
bradcray merged 1 commit intoBears-R-Us:mainfrom
jabraham17:fix-mypy-errors
Apr 13, 2026
Merged

Fix mypy errors caused by new version of mypy#5485
bradcray merged 1 commit intoBears-R-Us:mainfrom
jabraham17:fix-mypy-errors

Conversation

@jabraham17
Copy link
Copy Markdown
Contributor

@jabraham17 jabraham17 commented Apr 8, 2026

Fixes mypy errors caused by new versions of mypy, which showed up as:

arkouda/numpy/pdarraysetops.py:519: error: Item "Strings" of "pdarray | Strings | Categorical" has no attribute "max_bits"  [union-attr]
arkouda/numpy/pdarraysetops.py:519: error: Item "Categorical" of "pdarray | Strings | Categorical" has no attribute "max_bits"  [union-attr]
arkouda/numpy/pdarraysetops.py:520: error: Item "Strings" of "pdarray | Strings | Categorical" has no attribute "max_bits"  [union-attr]
arkouda/numpy/pdarraysetops.py:520: error: Item "Categorical" of "pdarray | Strings | Categorical" has no attribute "max_bits"  [union-attr]
arkouda/numpy/pdarraysetops.py:525: error: Item "Strings" of "pdarray | Strings | Categorical" has no attribute "max_bits"  [union-attr]
arkouda/numpy/pdarraysetops.py:525: error: Item "Categorical" of "pdarray | Strings | Categorical" has no attribute "max_bits"  [union-attr]

…operly

Signed-off-by: Jade Abraham <jade.abraham@hpe.com>
Copy link
Copy Markdown
Contributor

@bradcray bradcray left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thanks! Note that I edited the OP to capture the mypy errors that these changes address.

@bradcray bradcray added this pull request to the merge queue Apr 13, 2026
Merged via the queue into Bears-R-Us:main with commit 4adda5e Apr 13, 2026
19 checks passed
@jabraham17 jabraham17 deleted the fix-mypy-errors branch April 14, 2026 00:01
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.

2 participants