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