@darrylong are we planning to have a tutorial on best practices for using this API service?

@tqtg Yes we should! I'll include a readme in the directory, also with the developer documentation?

that would be great! Let me see how we can combine this with our current serving API. In the meantime, maybe you could help with a tutorial/guide on using this with wsgi server too.

@darrylong I've updated to use Flask app as the default way to serve model. I think it's better than maintaining different versions. Let me know if we're good with this approach, then we can develop user guide accordingly.

@darrylong I've updated to use Flask app as the default way to serve model. I think it's better than maintaining different versions. Let me know if we're good with this approach, then we can develop user guide accordingly.

@tqtg Hey thanks for the changes, they look great to me. I'm okay with Flask being used as the default way.

Only thing is that do we want all users to install Flask in order to use Cornac?

@darrylong that’s a good point. Maybe we don’t have to, and only need to tell users to install Flask as needed

@darrylong that’s a good point. Maybe we don’t have to, and only need to tell users to install Flask as needed

@tqtg alright! i was thinking maybe we can remove the dependency check from setup.py, and add a check in the serving portion instead. Will the be fine?

Yes. We can do that

LGTM

@tqtg Added the changes as discussed. Please have a look and feel free to edit or merge. Thanks!