yuruofeifei/mms

MXNet Model Serving

mms's Issues

Small suggestions

1. pull_model is only used in ModelService. It is a model management function more than a real prediction utility function. We can put it under ModelService class.
2. Currently pull_model only pulls 'MXNetModel' model, which is for resnet-18 specifically. We may want to change its interface to accept any model. This might also require another command line arguments, such as 'method', to specify the model class used for each model.
3. Follow up 2, we can consider putting a Base class for model, which only does fundamental preprocess, prediction and postprocess. If user doesn't need any extra preprocess/postprocess, they don't need to specify the 'method' argument of CLT.

pip install failed on macOS

c4b301cf6ca3:Downloads steroche$ pip install mms
Collecting mms
Downloading mms-0.1rc2.tar.gz (13.3MB)
100% |████████████████████████████████| 13.3MB 20kB/s
Complete output from command python setup.py egg_info:
Traceback (most recent call last):
File "", line 1, in
File "/private/var/folders/_5/vtwyq04d4zd3r54vybhc6v2n8qgblt/T/pip-build-g22lt675/mms/setup.py", line 3
print find_packages(exclude=["tests", "build", "app", "models", "dist", "mms.egg-info"])
^
SyntaxError: invalid syntax

----------------------------------------

Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/_5/vtwyq04d4zd3r54vybhc6v2n8qgblt/T/pip-build-g22lt675/mms/
c4b301cf6ca3:Downloads steroche$ python --version
Python 3.6.2 :: Anaconda, Inc.
c4b301cf6ca3:Downloads steroche$ pip install --upgrade pip
Requirement already up-to-date: pip in /Users/steroche/anaconda3/lib/python3.6/site-packages
c4b301cf6ca3:Downloads steroche$ pip install --upgrade setuptools
Requirement already up-to-date: setuptools in /Users/steroche/anaconda3/lib/python3.6/site-packages

Use MXNet Ndarray instead of numpy in utils

pip install mms returns No matching distribution found for mms

username@computer:/working/directory$ pip install mms
Collecting mms
Could not find a version that satisfies the requirement mms (from versions: )
No matching distribution found for mms