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:
[email protected]
With regards,
Apache Git Services