amplab / velox-modelserver

http://amplab.github.io/velox-modelserver/
Apache License 2.0
110 stars 26 forks source link

Avoid having to edit Velox core source code when adding a model #75

Closed dcrankshaw closed 9 years ago

dcrankshaw commented 9 years ago

Adding a new Model implementation to Velox is really gross. It requires adding a case statement on the type name, as well as recompiling with the new implementation. It would be great if we could somehow make velox-core a dependency for model implementations instead.

dcrankshaw commented 9 years ago

Fixed by #76