Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add an HTTP server for hosting of ONNX models #806

Merged
merged 237 commits into from
May 1, 2019

Conversation

tmccrmck
Copy link
Contributor

@tmccrmck tmccrmck commented Apr 10, 2019

  • Initial PR for feed back

  • Full integration tests hopefully implemented in the next week

  • not included but planned: HTTPS, gRPC, features like multiple models, batch, etc

tmccrmck and others added 30 commits March 1, 2019 10:36
Using https instead of ssh to add vcpkg as submodule
Adds Boost dependencies and basic HTTP server
* Add initial test which is failing linking with no main

* Adds test_main to get hosting tests working

* Deletes useless add_executable line
@pranavsharma
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 9 pipeline(s).

tmccrmck and others added 7 commits April 27, 2019 13:58
* Gets Boost compiling on Windows

* Fix integer conversion and comparison problems

* Use size_t in converter_tests instead of int

* Fix hosting integration tests on Windows

* Removes checks for port because it's an unsigned short

* Fixes comparison between signed and unsigned data types

* Pip install protobuf and numpy
pranavsharma
pranavsharma previously approved these changes Apr 30, 2019
@pranavsharma
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 9 pipeline(s).

* Changes minimum request size to 10MB to support all models in ONNX Model Zoo
Copy link
Contributor

@pranavsharma pranavsharma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's add memory leak check as well as part of CI.

@pranavsharma
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 9 pipeline(s).

@pranavsharma pranavsharma merged commit 1978b3c into microsoft:master May 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants