Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Fix reshape interoperability test (#17155) #17346

Closed
wants to merge 4 commits into from

Conversation

ChaiBapchya
Copy link
Contributor

  • fix reshape interoperability test

  • fix for scipy import

Description

To fix the numpy.decorator issue in #17286

@haojin2 @frankfliu @reminisce

* fix reshape interoperability test

* fix for scipy import
@@ -26,11 +26,11 @@ h5py==2.8.0rc1
mock==2.0.0
nose==1.3.7
nose-timer==0.7.3
numpy>1.16.0,<2.0.0
numpy>1.16.0,<1.18.0
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is there a restriction on the numpy version? I think it has been fixed in master.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I was told it was this PR's fix that helped resolve the issue #17154

Copy link
Contributor

@leezu leezu left a comment

Choose a reason for hiding this comment

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

matplotlib 2.2.4 has requirement python-dateutil>=2.1, but you'll have python-dateutil 1.5 which is incompatible needs fixing

@ChaiBapchya
Copy link
Contributor Author

ChaiBapchya commented Jan 23, 2020

Weird thing

https://github.com/ChaiBapchya/incubator-mxnet/blob/60dd6bd0dc7b902df50b3c8047d13f3c16647050/ci/docker_cache_requirements#L22

On the same branch python-dateutil==2.7.2 has been pinned... not sure why it doesn't pick that up.

@leezu leezu mentioned this pull request Jan 24, 2020
@ChaiBapchya
Copy link
Contributor Author

Closing in favor of #17426

@ChaiBapchya ChaiBapchya deleted the numpy_decorator_fix branch January 24, 2020 05:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants