apeforest commented on a change in pull request #11952: [MXNET-707] Add unit 
test for mxnet to coreml converter
URL: https://github.com/apache/incubator-mxnet/pull/11952#discussion_r223938744
 
 

 ##########
 File path: ci/travis/install.sh
 ##########
 @@ -22,5 +22,5 @@ export HOMEBREW_NO_AUTO_UPDATE=1
 
 if [ ${TRAVIS_OS_NAME} == "osx" ]; then
     brew install opencv
-    python -m pip install --user nose numpy cython scipy requests mock 
nose-timer nose-exclude
+    python -m pip install --user nose numpy cython scipy requests mock 
nose-timer nose-exclude mxnet-to-coreml
 
 Review comment:
   @marcoabreu  The purpose of this CI is not to validate the coreml package 
but to validate that the coreml package on PyPi can work with the latest MXNet 
release. There had been an issue earlier that breaks customer flow because 
coreml package was not up-to-date on PyPI. Without this validation in CI, the 
same problem could happen again in the future. In addition, mxnet-to-coreml 
converter package is in maintenance mode so I do not foresee a frequent update 
in this package.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to