mmpose icon indicating copy to clipboard operation
mmpose copied to clipboard

[Fix] replace np.int with int

Open jin-s13 opened this issue 2 years ago • 1 comments
trafficstars

Motivation

numpy 1.24.0 removes np.bool, np.float, np.int. To fix it, I replace np.int (and some np.int64) with np.int32.

Modification

BC-breaking (Optional)

Use cases (Optional)

Checklist

Before PR:

  • [ ] I have read and followed the workflow indicated in the CONTRIBUTING.md to create this PR.
  • [ ] Pre-commit or linting tools indicated in CONTRIBUTING.md are used to fix the potential lint issues.
  • [ ] Bug fixes are covered by unit tests, the case that causes the bug should be added in the unit tests.
  • [ ] New functionalities are covered by complete unit tests. If not, please add more unit tests to ensure correctness.
  • [ ] The documentation has been modified accordingly, including docstring or example tutorials.

After PR:

  • [ ] CLA has been signed and all committers have signed the CLA in this PR.

jin-s13 avatar Dec 27 '22 07:12 jin-s13

Same problems exist in xtcocotools, I will fix xtcocotools. We may need to upgrade xtcocotools afterwards.


edit:

We need to upgrade xtcocotools==1.13

jin-s13 avatar Dec 27 '22 07:12 jin-s13

Hello, I'd like to kindly follow up on this pull request. I tried downgrading numpy to 1.20.0 but then I get other errors from xtocotools. Thanks so much!

dancingdarwin avatar Feb 17 '23 07:02 dancingdarwin