Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[CI Problem] Android build failing with numpy error #13647

Closed
lhutton1 opened this issue Dec 20, 2022 · 4 comments
Closed

[CI Problem] Android build failing with numpy error #13647

lhutton1 opened this issue Dec 20, 2022 · 4 comments
Labels
type:ci Relates to TVM CI infrastructure

Comments

@lhutton1
Copy link
Contributor

This is failing due to the recently deprecated numpy value np.bool in the 1.9.1 release of mxnet. From the 1.24.0 release notes (https://numpy.org/doc/stable/release/1.24.0-notes.html#expired-deprecations):

The deprecation for the aliases np.object, np.bool, np.float, np.complex, np.str, and np.int is expired 
(introduces NumPy 1.20). Some of these will now give a FutureWarning in addition to raising an error 
since they will be mapped to the NumPy scalars in the future.
@lhutton1 lhutton1 added needs-triage PRs or issues that need to be investigated by maintainers to find the right assignees to address it type:ci Relates to TVM CI infrastructure labels Dec 20, 2022
@ashutosh-arm
Copy link
Contributor

Affected PRs:
#13637
#13644
#13643

lhutton1 added a commit to lhutton1/tvm that referenced this issue Dec 20, 2022
@lhutton1
[CI] Fix android build by constraining numpy version
17f47bd 
Temporarily constrain the version of numpy to workaround the
deprecated value used in mxnet. See apache#13647.

Change-Id: Ib271c223447c76b855fe35cc8a1e77411a3fa441
@lhutton1 lhutton1 mentioned this issue Dec 20, 2022
Merged
ashutosh-arm pushed a commit that referenced this issue Dec 20, 2022
@lhutton1
[CI] Fix android build by constraining numpy version (#13648)
26a205c 
Temporarily constrain the version of numpy to workaround the
deprecated value used in mxnet. See #13647.

Change-Id: Ib271c223447c76b855fe35cc8a1e77411a3fa441
@lhutton1
Copy link
Contributor Author

#13648 was merged as a temporary workaround, there likely needs to be a fix in MXNet to fix the compatibility issue. We can revert #13648 when it is resolved.

@lhutton1
Copy link
Contributor Author

Reported in apache/mxnet#21165

@lhutton1 lhutton1 removed the needs-triage PRs or issues that need to be investigated by maintainers to find the right assignees to address it label Feb 7, 2023
fzi-peccia pushed a commit to fzi-peccia/tvm that referenced this issue Mar 27, 2023
@lhutton1 @fzi-peccia
[CI] Fix android build by constraining numpy version (apache#13648)
7e2b8dd 
Temporarily constrain the version of numpy to workaround the
deprecated value used in mxnet. See apache#13647.

Change-Id: Ib271c223447c76b855fe35cc8a1e77411a3fa441
mikeseven pushed a commit to mikeseven/tvm that referenced this issue Sep 27, 2023
@lhutton1
[CI] Fix android build by constraining numpy version (apache#13648)
2891931 
Temporarily constrain the version of numpy to workaround the
deprecated value used in mxnet. See apache#13647.

Change-Id: Ib271c223447c76b855fe35cc8a1e77411a3fa441
@lhutton1
Copy link
Contributor Author

Closing now that mxnet is archived and being removed as a dependency from TVM.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type:ci Relates to TVM CI infrastructure
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lhutton1 @ashutosh-arm