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

Updated configuration.md for missing config properties #337

Merged
merged 13 commits into from
May 21, 2020

Conversation

shivamshriwas
Copy link
Contributor

Description

Added missing config properties in configuration.md
Added an note on use of enable_envvars_config.
Modifed MODEL_SERVER_HOME env variable to TS_MODEL_SERVER_HOME to make it inline with other config properties which can be set using env vars.

Fixes #204

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • This change requires a documentation update

Feature/Issue validation/testing

  • Test A
    set enable_envvars_config true in config.properties
    Use env variable to set config property. (refer configuration.md)
    Observe use of env variable in torchserve instance.

  • UT/IT execution results

  • Logs

  • [
    issue_204_torchserve_sanity_log.txt

Checklist:

] Have you added tests that prove your fix is effective or that this feature works?

  • New and existing unit tests pass locally with these changes?
  • Has code been commented, particularly in hard-to-understand areas?
  • Have you made corresponding changes to the documentation?

@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-cpu
  • Commit ID: ba630c5
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

- removes ambiguity
- follows naming convention
@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-gpu
  • Commit ID: ba630c5
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-cpu
  • Commit ID: acd2269
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-gpu
  • Commit ID: acd2269
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@chauhang chauhang requested review from mycpuorg and removed request for fbbradheintz May 16, 2020 21:31
@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-cpu
  • Commit ID: d8ca5e1
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-gpu
  • Commit ID: d8ca5e1
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@shivamshriwas shivamshriwas linked an issue May 19, 2020 that may be closed by this pull request
@maaquib maaquib requested review from maaquib and dhanainme May 19, 2020 21:06
@maaquib maaquib added this to the v0.1.1 milestone May 19, 2020
Copy link
Collaborator

@maaquib maaquib left a comment

Choose a reason for hiding this comment

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

Running into the following exception when testing this.
Steps for reproduction:

> cat /tmp/serve/config.properties
debug=true
model_store=/tmp/model-store
model_server_home=/tmp/model-server-home
max_request_size=1000000
max_response_size=1

> torchserve --start --ts-config config.properties

> curl -X POST "http://127.0.0.1:8081/models?url=https://torchserve.s3.amazonaws.com/mar_files/squeezenet1_1.mar"
{
  "status": "Model \"squeezenet1_1\" registered"
}

> curl -X PUT "http://0.0.0.0:8081/models/squeezenet1_1?min_worker=3"
{
  "status": "Processing worker updates..."
}
2020-05-20 00:07:23,814 [INFO ] epollEventLoopGroup-3-1 org.pytorch.serve.wlm.ModelManager - Model squeezenet1_1 loaded.
2020-05-20 00:07:26,880 [DEBUG] epollEventLoopGroup-3-2 org.pytorch.serve.wlm.ModelManager - updateModel: squeezenet1_1, count: 3
2020-05-20 00:07:26,886 [DEBUG] W-9000-squeezenet1_1_1.0 org.pytorch.serve.wlm.WorkerThread - W-9000-squeezenet1_1_1.0 State change null -> WORKER_STARTED
2020-05-20 00:07:26,886 [DEBUG] W-9000-squeezenet1_1_1.0 org.pytorch.serve.wlm.WorkerThread - W-9000-squeezenet1_1_1.0 State change null -> WORKER_STARTED
2020-05-20 00:07:26,886 [DEBUG] W-9000-squeezenet1_1_1.0 org.pytorch.serve.wlm.WorkerThread - W-9000-squeezenet1_1_1.0 State change null -> WORKER_STARTED
2020-05-20 00:07:26,888 [INFO ] W-9000-squeezenet1_1_1.0 org.pytorch.serve.wlm.WorkerThread - Connecting to: /tmp/.ts.sock.9000
2020-05-20 00:07:26,888 [INFO ] W-9000-squeezenet1_1_1.0 org.pytorch.serve.wlm.WorkerThread - Connecting to: /tmp/.ts.sock.9000
2020-05-20 00:07:26,888 [INFO ] W-9000-squeezenet1_1_1.0 org.pytorch.serve.wlm.WorkerThread - Connecting to: /tmp/.ts.sock.9000
2020-05-20 00:07:26,894 [ERROR] W-9000-squeezenet1_1_1.0 org.pytorch.serve.wlm.WorkerThread - Backend worker error
org.pytorch.serve.wlm.WorkerInitializationException: Failed to connect to worker.
>-------at org.pytorch.serve.wlm.WorkerThread.connect(WorkerThread.java:289)
>-------at org.pytorch.serve.wlm.WorkerThread.run(WorkerThread.java:120)
>-------at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)
>-------at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
>-------at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
>-------at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
>-------at java.base/java.lang.Thread.run(Thread.java:834)
Caused by: java.io.FileNotFoundException
>-------at io.netty.channel.unix.Errors.throwConnectException(Errors.java:110)
>-------at io.netty.channel.unix.Socket.connect(Socket.java:257)
>-------at io.netty.channel.epoll.AbstractEpollChannel.doConnect0(AbstractEpollChannel.java:732)
>-------at io.netty.channel.epoll.AbstractEpollChannel.doConnect(AbstractEpollChannel.java:717)
>-------at io.netty.channel.epoll.EpollDomainSocketChannel.doConnect(EpollDomainSocketChannel.java:87)
>-------at io.netty.channel.epoll.AbstractEpollChannel$AbstractEpollUnsafe.connect(AbstractEpollChannel.java:559)
>-------at io.netty.channel.DefaultChannelPipeline$HeadContext.connect(DefaultChannelPipeline.java:1366)
>-------at io.netty.channel.AbstractChannelHandlerContext.invokeConnect(AbstractChannelHandlerContext.java:545)
>-------at io.netty.channel.AbstractChannelHandlerContext.connect(AbstractChannelHandlerContext.java:530)
>-------at io.netty.channel.ChannelOutboundHandlerAdapter.connect(ChannelOutboundHandlerAdapter.java:47)
>-------at io.netty.channel.AbstractChannelHandlerContext.invokeConnect(AbstractChannelHandlerContext.java:545)
>-------at io.netty.channel.AbstractChannelHandlerContext.connect(AbstractChannelHandlerContext.java:530)
>-------at io.netty.channel.AbstractChannelHandlerContext.connect(AbstractChannelHandlerContext.java:512)
>-------at io.netty.channel.DefaultChannelPipeline.connect(DefaultChannelPipeline.java:1024)
>-------at io.netty.channel.AbstractChannel.connect(AbstractChannel.java:259)
>-------at io.netty.bootstrap.Bootstrap$3.run(Bootstrap.java:252)
>-------at io.netty.util.concurrent.AbstractEventExecutor.safeExecute(AbstractEventExecutor.java:163)
>-------at io.netty.util.concurrent.SingleThreadEventExecutor.runAllTasks(SingleThreadEventExecutor.java:404)
>-------at io.netty.channel.epoll.EpollEventLoop.run(EpollEventLoop.java:309)
>-------at io.netty.util.concurrent.SingleThreadEventExecutor$5.run(SingleThreadEventExecutor.java:884)
>-------at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)
>-------... 1 more

docs/configuration.md Outdated Show resolved Hide resolved
docs/configuration.md Outdated Show resolved Hide resolved
docs/configuration.md Outdated Show resolved Hide resolved
@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-cpu
  • Commit ID: b3f3db2
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-gpu
  • Commit ID: b3f3db2
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@shivamshriwas shivamshriwas self-assigned this May 20, 2020
@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-cpu
  • Commit ID: e5184ca
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-gpu
  • Commit ID: e5184ca
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@shivamshriwas
Copy link
Contributor Author

@shivamshriwas Can you also take a look at the min_worker scaling issue that I reported. If it is not related to your change, I'll create a separate issue. Thanks.

@maaquib
the issue here is not due to changes in this PR , its debug mode specific which is not intended to use as steps followed.
debug option is developer specific property hence as discussed removed from main doc.
issue #215 will be taking care of adding documentation around it.

@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-cpu
  • Commit ID: 5b8fe71
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-gpu
  • Commit ID: 5b8fe71
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-cpu
  • Commit ID: 29dfd93
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-gpu
  • Commit ID: 29dfd93
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

… issue_204

# Conflicts:
#	frontend/server/src/main/java/org/pytorch/serve/util/ConfigManager.java
@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-cpu
  • Commit ID: 3456e29
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-gpu
  • Commit ID: 3456e29
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-cpu
  • Commit ID: 1af405b
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

Copy link
Collaborator

@maaquib maaquib left a comment

Choose a reason for hiding this comment

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

Verified that this is working as expected after removing debug. The debug flag documentation has been removed from here and will be added to a seprate Developer documentation as per #215

@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-gpu
  • Commit ID: 1af405b
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@maaquib maaquib merged commit 3343b14 into staging_0_1_1 May 21, 2020
@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-cpu
  • Commit ID: 1af405b
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-gpu
  • Commit ID: 1af405b
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

mycpuorg added a commit that referenced this pull request Jun 9, 2020
* fixed resnet152 custom handler to work with GPU

* issue template

* Moving issue templates to root folder per pull request template

* adding feature request template

* Minor changes related to sections and names.

* Update feature_template.md

* Update bug_template.md

* doc template

* minor changes

* Minor changes to upload custom model or handler

* issue templates (#273)

* issue template

* Moving issue templates to root folder per pull request template

* adding feature request template

* Minor changes related to sections and names.

* Update feature_template.md

* Update bug_template.md

* doc template

* minor changes

* Minor changes to upload custom model or handler

Co-authored-by: mycpuorg <[email protected]>

* Fixes #205 - Docker install not finding GPUs (#262)

Invokes docker with  --runtime=nvidia for GPU
    Added the option to specify the GPU / specific GPU ids in start.sh
    Fixed the documentation for start.sh script
    Fixed the JDK version in Dockerfile.gpu

Co-authored-by: Aaqib <[email protected]>

* Fix pylint issue (#163)

* pylint fixes and code cleanup

* UT fixes, pylint updates per latest version and updated sanity script to run pylint

* fixed pylint and wrapt installation issue

* freeze pylint version to 2.4.4 due to bug in version 2.5

* removed duplicate statement to install pylint

Co-authored-by: Aaqib <[email protected]>

* Updated exception handling for worker threads shutdown. (#172)

* updated code to now log errors only in case when exception is not genreated by unregister api call

* java formatting

* fixed illigalstateexception

* removed the unrequired handling for illegal monitor state exception

Co-authored-by: dhaniram kshirsagar <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* changes for Homebrew  issue 98 (#126)

* changes for Homebrew  issue 98

* adding changes for path in benchmark.py

* CI Build corrected

* benchmark changes

corrected path

* changed PATH for homebrew

* Update Readme

Made changes for homebrew installation and instruction for GPU based machine instruction for install_dependencies.sh

* Updated Readme.md

removed unnecessory  lines

* corrected indentation

corrected indentation for  new lines added

* Update Readme.md fil

added separate commands for CPU and GPU based

Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: harshbafna <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Harsh Bafna <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* added changes to clean model loaded through URL from model store on unregisted

* added UTs

* Moving the plugins SDK to org.pytorch. Added plugins example. (#261)

* Moved plugins SDK to org.pytorch

* Updating the repo for publishing plugins SDK

* Enabled bintray maven repo temporarily

* Reset tests

* Added GPG plugin to pom file to sign the plugins SDK

* Update the SDK version

* Removed bintray repo

* Modify the buildspec to not deploy the SDK

Co-authored-by: Vamshidhar Dantu <[email protected]>
Co-authored-by: Aaqib <[email protected]>
Co-authored-by: mycpuorg <[email protected]>

* Added model zoo (#299)

Co-authored-by: mycpuorg <[email protected]>
Co-authored-by: Brad Heintz <[email protected]>

* Removed snapshot dependency on model (#152)

* Snapshot is now independent of model

* fixed checkstyle issue

* fixed formatting

* doc review: configuration and custom_service

* addressed review comments

* fixed incorrect merge

* fixed review comments and corrected the error messages

* adding remaining fixes for string constants

Co-authored-by: harshbafna <[email protected]>
Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>
Co-authored-by: Harsh Bafna <[email protected]>

* Regression Tests Suite - AWS Code Build Infra - Initial Commit for #57 (#285)

* Regression Tests Suite - AWS Code Build Infra - Initial Commit for #57

* Script to clone Pytorch Repo & Run Post Man Scripts
* Basic Tests for Inference & Managment APIs
* AWS Code Build buildspec.yml

* Refactor postman tests, add empty postman collection for todos

* Add a readme for adding tests

* Add more management tests

* Fix typo

* Snapshot Regression tests - #57

* Fix typo

* Comment out snapshot test for now

* Weird typos are showing up, try saving everything

* Need to manually save every request in postman...

* When reregister don't use s3

* More workarounds for rereg

* Fix test messages

* Remove test for empty reqs

* Change get to post

* Fix pytest for subprocess - #57

* Enable pytest - #57

* Inference API Testcases / Rename Management Testcases  #57

* Wipe Model MARs in Modelstore between tests #54

* Fix Inference Endpoint Execution #54

* Fix file paths - #57

* Fix Initial Workers #57

* Fix Inference Assertions #57

* Fix DenseNet Inference #57

* Fix Densenet Register #54

* Redirect Logs to File #54

* Fix Test Execution Log #54

* README, Build Badge #57

* Format ReadMe #57

* Update README - Add Instruction to add new tests / Run tests locally

* Update README

* Update branch for build Codebuild Badge

Co-authored-by: alexwong <[email protected]>
Co-authored-by: Aaqib <[email protected]>
Co-authored-by: mycpuorg <[email protected]>
Co-authored-by: Ubuntu <[email protected]>

* Issue 312 install from src (#322)

* documentation: fixed issues in installation and quick start in README.md

* documentation: Reverting conda related doc changes from PR#286

Fixes #297

* remove minus y (#311)

update pip install instructions

* Update instructions for installation from source

This commit aims to address Issue #312

Tested on a fresh EC2 P3.2 instance:
* Followed instructions under
https://github.com/pytorch/serve#install-torchserve-for-development
* Started the server manually from command line
* Ran a ping query

Received a response from server with "Healthy" status

Signed-off-by: Manoj Rao <[email protected]>

* add separate files for installation

one each for linux and macos

* update README and remove relative path from script

* Install script - Dont uninstall if not previously installed - #132

* explicitly build the source code with gradle prior to install

* remove upgrade instructions in favor of clean reinstall from source

* update instructions to contain the correct script name

Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaqib Ansari <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>
Co-authored-by: Manoj Rao <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>

* fix for macos script (#338)

* Move Issue Template to .github dir

* Updated sanity script to use install_from_src_ubuntu script (#350)

* updated sanity script to use install_from_src_ubuntu script

* added java version check

* fixed issue with if statement

* fixed variable name

* fixed repeated cd command

* updated artifacts

* removed artifacts from buildspec

* Corrected HTTP response code for Model version does not exists exception (#282)

* issue 196 patch

* Added openapi missing responses. - incorporated review comments

* updated method signature after resolving merge conflicts

Co-authored-by: Harsh Bafna <[email protected]>
Co-authored-by: Aaqib <[email protected]>
Co-authored-by: Shivam Shriwas <[email protected]>
Co-authored-by: harshbafna <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Corrected paths in command and added notes (#354)

Co-authored-by: dhanainme <[email protected]>

* Refactored UT for better reporting (#159)

* refactored UTs

* upgraded testng version and updated testcase names

* added testng.xml to handle sequential execution of test classes

* added waveglow example

* Updated readme for waveglow example

* fixed failing UTs after merging master

* removed unintentionally pushed waveglow example

* fixed failing test case

* moved set-default to utils and added couple of more add cases for default version

* java formatting

* corrected set default negative test case

* fixed java formatting

Co-authored-by: Aaqib <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Huggingface Transformers generalization (#302)

* Scripts to seriliaze a Bert model using jit.trace and a handler for Bert model

* modified to include mapping of predicitons and decoding received text

* move back the original text

* moved the index_to_name to bert folder

* Transformers/ Bert serving with Torchserve readme

* deleting an unused dictionary

* just cleaning some of the commented out parts

* cleaning up

* Cleaning up

* uncommenting the logging and cleaning  the comments

* Download and save the checkpoints, config file and vocab of  the BERT from Transformers for demonstration purposes

* mapping of predicitons to labels

* a custom handler for transformers.

* README.

* sample of input text.

* modifying the descriptions

* modifying the descriptions

* adding comments

* adding commnets

* Fromatting modifications

* modified comments

* added the citations

* adding hugginface as the type of transformer model we are referring to

* renaming the folder to huggingface_transformers

* Removing the bert folder

* simple modification

* modifed the script to use a setup_config.json for loading and saving models

* added the section for transfomer_handler_generalized

* the generalized handler

* set_up config intialization

* edited for transfomer generalization

* modified to use just input_ids as inputs for models

* modified for question answering

* modified to include the number of labels

* corrected typos

* modifed variable names

* edited for clarity

* modifed variable names, simplified the initlialize in the handler class, removed the warnning from QA case

* modifed values for QA

* modifed the variable names

* added comments

* modified to add clarity

* add comment in preprocessing of QA

* added warning for the case mode is not selected properly and combined constructor and initlaize fucntion

* clean up

* clean up

* combined constructor and initliaze func

* combined constructor and initliaze func

* Separated the constructor and initliaze func

* Modified to remove the classification part and update with generalized version

* resolved the coflict and separated the costructor and initliaze func

* removed the handler for classification case as the generalized version handles the case

Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: mycpuorg <[email protected]>

* added waveglow example (#258)

* added waveglow example

* Updated readme for waveglow example

* enhanced example to remove runtime checkpoint download

* updated handler to return the audio file content instead of dumping it to tmp dir

* minor change to handler

* added pipefail

* added python snippet to run inference

* updated readme with know issue with the example

Co-authored-by: Aaqib <[email protected]>

* made build suite compatible with latest gradle (#348)

Co-authored-by: dhanainme <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* moved env section after context

* Deleting redundant ISSUE_TEMPLATE at root dir

* Fix typo in scripts/install_from_src_macos

* Corrected  OpenApi description  (#328)

* Correct query parameter name (Registration Api)in OpenApi description

* removed non existing path parameter

Co-authored-by: Aaqib <[email protected]>

* Regression suite fixes and added test cases (#344)

* added test model zoo test cases
updated regression script

* Updated Readme

* update -y flag for install

* Updated docker command as per new imagename and tag generated in stagging branch

* Fix sudo usage in regression test script

* - addressed review comments
- reverted changes related to sudo in regression script.

* added https test collection

* - updated failed test cases on staging branch
- added environment and updated newman cmd  with --insecure for https

* updated script

* updated pytest

* update curl check for serve up

* Revert "update curl check for serve up"

This reverts commit 0047eae.

* Revert "update curl check for serve up"

This reverts commit 0047eae.

- update model zoo
- added resnet test case

* added workers to resnet model

* Update readme to run test suite on desired branch

* enhanced inference test cases to driven by datafile

* Updated readme

Co-authored-by: Dhanasekar Karuppasamy <[email protected]>
Co-authored-by: Aaqib <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Fix typo in mnist readme (#330)

Co-authored-by: Aaqib <[email protected]>

* Fixed typo in ab installation. (#327)

* documentation: fixed issues in installation and quick start in README.md

* documentation: Reverting conda related doc changes from PR#286

Fixes #297

* remove minus y (#311)

update pip install instructions

* Fixed typo in ab installation.

Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaqib Ansari <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>

* Setup Nightly Build for TorchServe (#371)

* Add Branch Flag for Nightly CodeBuild #57

* Update ENV Variable for Nightly Build Name

* Nigtly Build - Fix FileName for Build Artifacts

* Update README for accessing test logs

* Whitspace changes

* Build new images for build - README Instruction

* Fix for intermittent gradle build failure (#352)

* Fix for intermittent null pointer exception

* Fixed comments

Co-authored-by: dhanainme <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* documentation: Remove redundant curl options in example (#336)

* documentation: fixed issues in installation and quick start in README.md

* documentation: Reverting conda related doc changes from PR#286

Fixes #297

* remove minus y (#311)

update pip install instructions

* fix-doc

* fix remaining "-X POST"s

Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaqib Ansari <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>

* Corrected port number validation range (#304)

* Corrected port number validation range

* added UT for issue 291

* added ut dependency file formatting

* correct param in config_test_env.properties

* removed incorrect flags from test case config files

* added negative test cases

* removed properties file and changes in test case

Co-authored-by: Harsh Bafna <[email protected]>

* Fixes version number reporting for TorchServe (#360)

* Correct version and added vesion.txt for getting version info

* fixed description test cases and build issue

* Added snapshot test fixes
and resolved relative path issue for version.txt

* python formatting

* added version.txt in MANIFEST.in

* addressed review comments

* added version.txt for model archiver as well
- addressed review comments

* -fixed python UT
-added ts python UT run in sanity script

* removed blank lines from snapshot files

Co-authored-by: Aaqib <[email protected]>
Co-authored-by: Harsh Bafna <[email protected]>

* fixes installation script exits without error if no java present on machine (#376)

* fixes installation script exits without error if no java present on machine

* added changes for macos script

Co-authored-by: Aaqib <[email protected]>

* Dockerfile enhancements (#308)

* dockerfile now points to latest torchserve on pypi and added separate files for dev

* Minor changes

* Made build and run commands consistent

* Fixing formatting

* Consistency issue

* Enhanced Dockerfile for binary per #178 and other misc.

* Fixed gpu related issues in start script

* Corrected the heading font

* Added mar creation steps from container

* added buildkit support for dev docker files

* Fix for buildkit

* Restoring psutil and future as absence leads intermittent hangged build

* Fixed review comments for doc

* Fixed typo

Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* Updated configuration.md for missing config properties (#337)

* Updated configuration.md for missing parameters

* changes TS_MODEL_SNAPSHOT to MODEL_SNAPSHOT
- removes ambiguity
- follows naming convention

* addressed review comments.

* removed debug property

* updated configuration.md

Co-authored-by: Harsh Bafna <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* Adding missing document links to README (#379)

* Maven tests should output only errors (#326)

Co-authored-by: Vamshidhar Dantu <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* fixed code for build failure in python virtual environment (#341)

* fixed code for build failure in python virtual environment

* added exception missed on resolving conflicts

Co-authored-by: Aaqib <[email protected]>
Co-authored-by: Vamshidhar Dantu <[email protected]>

* updated object detector handler to use only default cuda device for torchvision version less then 0.6 (#265)

* updated object detector handler to use default cuda device for torchvision version 0.5.0

* updated version check logic

* Updated documentation for torchvision version dependency

* enhanced sanity script to validate all default handlers

* fixed sanity script

* added torchtext install step in sanity script

* fixed issue related to inference after restart with snapshot.

* moved torchtext to codebuild docker files

* added more log statements

* enhanced run inference function

* fixed register model function

* added unregister model function

* incorporated code review comments

* added back inference call for snapshot test

* fixed if block

Co-authored-by: Aaqib <[email protected]>

* Adding a barebones ec2 template to run torchserve on ubuntu (#301)

* Adding a barebones ec2 template to run torchserve on ubuntu

* Adding VPC with Private subnet

* Address review comments #1

* Address review comments #2

* Rebase for #365 (#389)

* Updated README.md for #364

See #364 for a deep description of the problem

* updated README.md

* updated README.md

Co-authored-by: Francesco Saverio Zuppichini <[email protected]>

* Document the docker settings for running in production mode (#388)

* Docker Production Options #234

* Docker Production Options - Typo Fixes / Fit & Finish  #234

* Typos - Docker Docs #234

* grammar check  - docker/README.md

Co-authored-by: Aaqib <[email protected]>

* grammar check  - docker/README.md

Co-authored-by: Aaqib <[email protected]>

* grammar check  - docker/README.md

Co-authored-by: Aaqib <[email protected]>

Co-authored-by: Aaqib <[email protected]>

* Huggingface transformers integration test (#382)

* Add Huggingface transformers example integration tests

* Add transformers intallation

* Fix a missing bracket

* move transformers installation to chain installations

Co-authored-by: Geeta Chauhan <[email protected]>

* Updated docker README.md (#401)

* Update README.md

* Fixed cd command

* Made source cloning steps explicit!!

* Fixed online editing mistake

Co-authored-by: Geeta Chauhan <[email protected]>

* Fixed response status message for inferencing on model with zero workers (#305)

* Changes for issue_29

* Changed error Messages

* changed message for scalling request

* changed defaultVersion string

* Corrected error message

* changed the code for best practice

* resolved review comment for version

* updated management api doc as per new responses in worker scale api

Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Harsh Bafna <[email protected]>
Co-authored-by: dhanainme <[email protected]>
Co-authored-by: Shivam Shriwas <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* made model server port tests independent (#396)

* Added backend profiling and fixed benchmark's install dependency scripts (#402)

* added backend profiling
* updated install dependency scripts to make them jmeter version independent
* updated readme
* Fix benchmarks/README typo and JMeter versioning issue
* updated documentation

Co-authored-by: Aaqib Ansari <[email protected]>

* Add sphinx directory and conf.py (#274)

* Add sphinx folder that contains files for docs build

* Delete .DS_Store

* Delete .DS_Store

* Add conf.py in docs/

* documentation: fixed issues in installation and quick start in README.md

* documentation: Reverting conda related doc changes from PR#286

Fixes #297

* remove minus y (#311)

update pip install instructions

* Update to the latest version of sphinx

* Add recommonmark to extensions

Add recommonmark to extensions so sphinx project has markdown support.
Configuration details here: https://www.sphinx-doc.org/en/master/usage/markdown.html

* Update source suffix to use latest recommonmark syntax

Updating to the syntax found here: https://www.sphinx-doc.org/en/master/usage/markdown.html

Co-authored-by: Aaron Markham <[email protected]>

* Remove old recommonmark parser syntax

* Pin sphinx version to work with latest version of recommonmark

Co-authored-by: Aaron Markham <[email protected]>

* Pin version latest version recommonmark

Co-authored-by: Aaron Markham <[email protected]>

* Delete theme file generated during build

* de-duplicate install package of psutil (#329)

The Dockerfile of GPU install package psutil duplicated.

* Remove pytorch-sphinx-theme files

* Update to match latest staging_0_1_1

* Update docker directory to match staging_0_1_1 branch

* Delete .DS_Store

* Add sphinxcontrib.katex and torch

Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaqib Ansari <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>
Co-authored-by: Mario Cho <[email protected]>
Co-authored-by: mycpuorg <[email protected]>

* Image classifier example readme fixes (#405)

* Changed the model name in torch.jit.save to be consistent with docs

* Remove extra handler flag and fix the torchservr start

* fix the doc

* fix the doc

Co-authored-by: mycpuorg <[email protected]>

* Porting memory fix changes to TorchServe (#422)

Co-authored-by: dhanainme <[email protected]>

* Benchmark documentation fix (#425)

* Fix Benchmark Documentation

* Documentaion fixes

* .circleci: Add initial circleci configuration (#431)

Signed-off-by: Eli Uriegas <[email protected]>

* Torchscript support huggingface transfrmers (#421)

* Add support for torchscript

* Add support for torchscript

* Add key values for torchscript support

* Add Torchscript suport

* Add Torchscript support

* Add Torchscript support in loading the models

* keep the example to QA

* modify to reflect Torchscript example?

* Add torchscript support guides

* Add fintued BERT for QA

* Add QA artifacts

* Add seqenece classification artifacts

* Token classification artifacts

* Correct typos

* Add max length to tokenizer

* clean up

* enhance the doc

* change setup to seq classification

* Add detail to torchscript support

* set the model for QA

* Add torchscript bert seq classification example

* Add  bert seq classification example

* Modify the packaging model command

* Modify torchscript packaging command

* clean up

* clean up

* correct spacing between list items

Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Integration test huggingface transfromers torchscript support (#433)

* made model server port tests independent

* Add integration tests for Huggingface transfromers with new handler that support torchscript

Co-authored-by: harshbafna <[email protected]>
Co-authored-by: mycpuorg <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Apachebench benchmark - cleanup script / documentation

* Revert "Apachebench benchmark - cleanup script / documentation"

This reverts commit c8f8682.

* Documentation updates for clarity on Eager mode / Torch scripted models (#439)

* Model Zoo updates for Eager Mode / Torchscripted models

* Update examples/README.md

Co-authored-by: Aaqib <[email protected]>

Co-authored-by: Aaqib <[email protected]>

* Apachebench benchmarking - Script / Documentation updates (#438)

* Apachebench benchmarking script - Script / Documentation updates

* Stop TS after benchmark completion.

Co-authored-by: harshbafna <[email protected]>
Co-authored-by: dhaniram-kshirsagar <[email protected]>
Co-authored-by: dhaniram kshirsagar <[email protected]>
Co-authored-by: Aaqib <[email protected]>
Co-authored-by: mycpuorg <[email protected]>
Co-authored-by: Harsh Bafna <[email protected]>
Co-authored-by: mjpsl <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Vamshidhar Dantu <[email protected]>
Co-authored-by: Vamshidhar Dantu <[email protected]>
Co-authored-by: Brad Heintz <[email protected]>
Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>
Co-authored-by: alexwong <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: dkasar-aws <[email protected]>
Co-authored-by: Shivam Shriwas <[email protected]>
Co-authored-by: Hamid Shojanazeri <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: shivamshriwas <[email protected]>
Co-authored-by: Takayoshi Nishida <[email protected]>
Co-authored-by: alvations <[email protected]>
Co-authored-by: Tzu-Wei Huang <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Francesco Saverio Zuppichini <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Jessica Lin <[email protected]>
Co-authored-by: Mario Cho <[email protected]>
Co-authored-by: Eli Uriegas <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
maaquib added a commit that referenced this pull request Jun 9, 2020
* fixed resnet152 custom handler to work with GPU

* issue template

* Moving issue templates to root folder per pull request template

* adding feature request template

* Minor changes related to sections and names.

* Update feature_template.md

* Update bug_template.md

* doc template

* minor changes

* Minor changes to upload custom model or handler

* issue templates (#273)

* issue template

* Moving issue templates to root folder per pull request template

* adding feature request template

* Minor changes related to sections and names.

* Update feature_template.md

* Update bug_template.md

* doc template

* minor changes

* Minor changes to upload custom model or handler

Co-authored-by: mycpuorg <[email protected]>

* Fixes #205 - Docker install not finding GPUs (#262)

Invokes docker with  --runtime=nvidia for GPU
    Added the option to specify the GPU / specific GPU ids in start.sh
    Fixed the documentation for start.sh script
    Fixed the JDK version in Dockerfile.gpu

Co-authored-by: Aaqib <[email protected]>

* Fix pylint issue (#163)

* pylint fixes and code cleanup

* UT fixes, pylint updates per latest version and updated sanity script to run pylint

* fixed pylint and wrapt installation issue

* freeze pylint version to 2.4.4 due to bug in version 2.5

* removed duplicate statement to install pylint

Co-authored-by: Aaqib <[email protected]>

* Updated exception handling for worker threads shutdown. (#172)

* updated code to now log errors only in case when exception is not genreated by unregister api call

* java formatting

* fixed illigalstateexception

* removed the unrequired handling for illegal monitor state exception

Co-authored-by: dhaniram kshirsagar <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* changes for Homebrew  issue 98 (#126)

* changes for Homebrew  issue 98

* adding changes for path in benchmark.py

* CI Build corrected

* benchmark changes

corrected path

* changed PATH for homebrew

* Update Readme

Made changes for homebrew installation and instruction for GPU based machine instruction for install_dependencies.sh

* Updated Readme.md

removed unnecessory  lines

* corrected indentation

corrected indentation for  new lines added

* Update Readme.md fil

added separate commands for CPU and GPU based

Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: harshbafna <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Harsh Bafna <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* added changes to clean model loaded through URL from model store on unregisted

* added UTs

* Moving the plugins SDK to org.pytorch. Added plugins example. (#261)

* Moved plugins SDK to org.pytorch

* Updating the repo for publishing plugins SDK

* Enabled bintray maven repo temporarily

* Reset tests

* Added GPG plugin to pom file to sign the plugins SDK

* Update the SDK version

* Removed bintray repo

* Modify the buildspec to not deploy the SDK

Co-authored-by: Vamshidhar Dantu <[email protected]>
Co-authored-by: Aaqib <[email protected]>
Co-authored-by: mycpuorg <[email protected]>

* Added model zoo (#299)

Co-authored-by: mycpuorg <[email protected]>
Co-authored-by: Brad Heintz <[email protected]>

* Removed snapshot dependency on model (#152)

* Snapshot is now independent of model

* fixed checkstyle issue

* fixed formatting

* doc review: configuration and custom_service

* addressed review comments

* fixed incorrect merge

* fixed review comments and corrected the error messages

* adding remaining fixes for string constants

Co-authored-by: harshbafna <[email protected]>
Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>
Co-authored-by: Harsh Bafna <[email protected]>

* Regression Tests Suite - AWS Code Build Infra - Initial Commit for #57 (#285)

* Regression Tests Suite - AWS Code Build Infra - Initial Commit for #57

* Script to clone Pytorch Repo & Run Post Man Scripts
* Basic Tests for Inference & Managment APIs
* AWS Code Build buildspec.yml

* Refactor postman tests, add empty postman collection for todos

* Add a readme for adding tests

* Add more management tests

* Fix typo

* Snapshot Regression tests - #57

* Fix typo

* Comment out snapshot test for now

* Weird typos are showing up, try saving everything

* Need to manually save every request in postman...

* When reregister don't use s3

* More workarounds for rereg

* Fix test messages

* Remove test for empty reqs

* Change get to post

* Fix pytest for subprocess - #57

* Enable pytest - #57

* Inference API Testcases / Rename Management Testcases  #57

* Wipe Model MARs in Modelstore between tests #54

* Fix Inference Endpoint Execution #54

* Fix file paths - #57

* Fix Initial Workers #57

* Fix Inference Assertions #57

* Fix DenseNet Inference #57

* Fix Densenet Register #54

* Redirect Logs to File #54

* Fix Test Execution Log #54

* README, Build Badge #57

* Format ReadMe #57

* Update README - Add Instruction to add new tests / Run tests locally

* Update README

* Update branch for build Codebuild Badge

Co-authored-by: alexwong <[email protected]>
Co-authored-by: Aaqib <[email protected]>
Co-authored-by: mycpuorg <[email protected]>
Co-authored-by: Ubuntu <[email protected]>

* Issue 312 install from src (#322)

* documentation: fixed issues in installation and quick start in README.md

* documentation: Reverting conda related doc changes from PR#286

Fixes #297

* remove minus y (#311)

update pip install instructions

* Update instructions for installation from source

This commit aims to address Issue #312

Tested on a fresh EC2 P3.2 instance:
* Followed instructions under
https://github.com/pytorch/serve#install-torchserve-for-development
* Started the server manually from command line
* Ran a ping query

Received a response from server with "Healthy" status

Signed-off-by: Manoj Rao <[email protected]>

* add separate files for installation

one each for linux and macos

* update README and remove relative path from script

* Install script - Dont uninstall if not previously installed - #132

* explicitly build the source code with gradle prior to install

* remove upgrade instructions in favor of clean reinstall from source

* update instructions to contain the correct script name

Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaqib Ansari <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>
Co-authored-by: Manoj Rao <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>

* fix for macos script (#338)

* Move Issue Template to .github dir

* Updated sanity script to use install_from_src_ubuntu script (#350)

* updated sanity script to use install_from_src_ubuntu script

* added java version check

* fixed issue with if statement

* fixed variable name

* fixed repeated cd command

* updated artifacts

* removed artifacts from buildspec

* Corrected HTTP response code for Model version does not exists exception (#282)

* issue 196 patch

* Added openapi missing responses. - incorporated review comments

* updated method signature after resolving merge conflicts

Co-authored-by: Harsh Bafna <[email protected]>
Co-authored-by: Aaqib <[email protected]>
Co-authored-by: Shivam Shriwas <[email protected]>
Co-authored-by: harshbafna <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Corrected paths in command and added notes (#354)

Co-authored-by: dhanainme <[email protected]>

* Refactored UT for better reporting (#159)

* refactored UTs

* upgraded testng version and updated testcase names

* added testng.xml to handle sequential execution of test classes

* added waveglow example

* Updated readme for waveglow example

* fixed failing UTs after merging master

* removed unintentionally pushed waveglow example

* fixed failing test case

* moved set-default to utils and added couple of more add cases for default version

* java formatting

* corrected set default negative test case

* fixed java formatting

Co-authored-by: Aaqib <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Huggingface Transformers generalization (#302)

* Scripts to seriliaze a Bert model using jit.trace and a handler for Bert model

* modified to include mapping of predicitons and decoding received text

* move back the original text

* moved the index_to_name to bert folder

* Transformers/ Bert serving with Torchserve readme

* deleting an unused dictionary

* just cleaning some of the commented out parts

* cleaning up

* Cleaning up

* uncommenting the logging and cleaning  the comments

* Download and save the checkpoints, config file and vocab of  the BERT from Transformers for demonstration purposes

* mapping of predicitons to labels

* a custom handler for transformers.

* README.

* sample of input text.

* modifying the descriptions

* modifying the descriptions

* adding comments

* adding commnets

* Fromatting modifications

* modified comments

* added the citations

* adding hugginface as the type of transformer model we are referring to

* renaming the folder to huggingface_transformers

* Removing the bert folder

* simple modification

* modifed the script to use a setup_config.json for loading and saving models

* added the section for transfomer_handler_generalized

* the generalized handler

* set_up config intialization

* edited for transfomer generalization

* modified to use just input_ids as inputs for models

* modified for question answering

* modified to include the number of labels

* corrected typos

* modifed variable names

* edited for clarity

* modifed variable names, simplified the initlialize in the handler class, removed the warnning from QA case

* modifed values for QA

* modifed the variable names

* added comments

* modified to add clarity

* add comment in preprocessing of QA

* added warning for the case mode is not selected properly and combined constructor and initlaize fucntion

* clean up

* clean up

* combined constructor and initliaze func

* combined constructor and initliaze func

* Separated the constructor and initliaze func

* Modified to remove the classification part and update with generalized version

* resolved the coflict and separated the costructor and initliaze func

* removed the handler for classification case as the generalized version handles the case

Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: mycpuorg <[email protected]>

* added waveglow example (#258)

* added waveglow example

* Updated readme for waveglow example

* enhanced example to remove runtime checkpoint download

* updated handler to return the audio file content instead of dumping it to tmp dir

* minor change to handler

* added pipefail

* added python snippet to run inference

* updated readme with know issue with the example

Co-authored-by: Aaqib <[email protected]>

* made build suite compatible with latest gradle (#348)

Co-authored-by: dhanainme <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* moved env section after context

* Deleting redundant ISSUE_TEMPLATE at root dir

* Fix typo in scripts/install_from_src_macos

* Corrected  OpenApi description  (#328)

* Correct query parameter name (Registration Api)in OpenApi description

* removed non existing path parameter

Co-authored-by: Aaqib <[email protected]>

* Regression suite fixes and added test cases (#344)

* added test model zoo test cases
updated regression script

* Updated Readme

* update -y flag for install

* Updated docker command as per new imagename and tag generated in stagging branch

* Fix sudo usage in regression test script

* - addressed review comments
- reverted changes related to sudo in regression script.

* added https test collection

* - updated failed test cases on staging branch
- added environment and updated newman cmd  with --insecure for https

* updated script

* updated pytest

* update curl check for serve up

* Revert "update curl check for serve up"

This reverts commit 0047eae.

* Revert "update curl check for serve up"

This reverts commit 0047eae.

- update model zoo
- added resnet test case

* added workers to resnet model

* Update readme to run test suite on desired branch

* enhanced inference test cases to driven by datafile

* Updated readme

Co-authored-by: Dhanasekar Karuppasamy <[email protected]>
Co-authored-by: Aaqib <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Fix typo in mnist readme (#330)

Co-authored-by: Aaqib <[email protected]>

* Fixed typo in ab installation. (#327)

* documentation: fixed issues in installation and quick start in README.md

* documentation: Reverting conda related doc changes from PR#286

Fixes #297

* remove minus y (#311)

update pip install instructions

* Fixed typo in ab installation.

Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaqib Ansari <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>

* Setup Nightly Build for TorchServe (#371)

* Add Branch Flag for Nightly CodeBuild #57

* Update ENV Variable for Nightly Build Name

* Nigtly Build - Fix FileName for Build Artifacts

* Update README for accessing test logs

* Whitspace changes

* Build new images for build - README Instruction

* Fix for intermittent gradle build failure (#352)

* Fix for intermittent null pointer exception

* Fixed comments

Co-authored-by: dhanainme <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* documentation: Remove redundant curl options in example (#336)

* documentation: fixed issues in installation and quick start in README.md

* documentation: Reverting conda related doc changes from PR#286

Fixes #297

* remove minus y (#311)

update pip install instructions

* fix-doc

* fix remaining "-X POST"s

Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaqib Ansari <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>

* Corrected port number validation range (#304)

* Corrected port number validation range

* added UT for issue 291

* added ut dependency file formatting

* correct param in config_test_env.properties

* removed incorrect flags from test case config files

* added negative test cases

* removed properties file and changes in test case

Co-authored-by: Harsh Bafna <[email protected]>

* Fixes version number reporting for TorchServe (#360)

* Correct version and added vesion.txt for getting version info

* fixed description test cases and build issue

* Added snapshot test fixes
and resolved relative path issue for version.txt

* python formatting

* added version.txt in MANIFEST.in

* addressed review comments

* added version.txt for model archiver as well
- addressed review comments

* -fixed python UT
-added ts python UT run in sanity script

* removed blank lines from snapshot files

Co-authored-by: Aaqib <[email protected]>
Co-authored-by: Harsh Bafna <[email protected]>

* fixes installation script exits without error if no java present on machine (#376)

* fixes installation script exits without error if no java present on machine

* added changes for macos script

Co-authored-by: Aaqib <[email protected]>

* Dockerfile enhancements (#308)

* dockerfile now points to latest torchserve on pypi and added separate files for dev

* Minor changes

* Made build and run commands consistent

* Fixing formatting

* Consistency issue

* Enhanced Dockerfile for binary per #178 and other misc.

* Fixed gpu related issues in start script

* Corrected the heading font

* Added mar creation steps from container

* added buildkit support for dev docker files

* Fix for buildkit

* Restoring psutil and future as absence leads intermittent hangged build

* Fixed review comments for doc

* Fixed typo

Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* Updated configuration.md for missing config properties (#337)

* Updated configuration.md for missing parameters

* changes TS_MODEL_SNAPSHOT to MODEL_SNAPSHOT
- removes ambiguity
- follows naming convention

* addressed review comments.

* removed debug property

* updated configuration.md

Co-authored-by: Harsh Bafna <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* Adding missing document links to README (#379)

* Maven tests should output only errors (#326)

Co-authored-by: Vamshidhar Dantu <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* fixed code for build failure in python virtual environment (#341)

* fixed code for build failure in python virtual environment

* added exception missed on resolving conflicts

Co-authored-by: Aaqib <[email protected]>
Co-authored-by: Vamshidhar Dantu <[email protected]>

* updated object detector handler to use only default cuda device for torchvision version less then 0.6 (#265)

* updated object detector handler to use default cuda device for torchvision version 0.5.0

* updated version check logic

* Updated documentation for torchvision version dependency

* enhanced sanity script to validate all default handlers

* fixed sanity script

* added torchtext install step in sanity script

* fixed issue related to inference after restart with snapshot.

* moved torchtext to codebuild docker files

* added more log statements

* enhanced run inference function

* fixed register model function

* added unregister model function

* incorporated code review comments

* added back inference call for snapshot test

* fixed if block

Co-authored-by: Aaqib <[email protected]>

* Adding a barebones ec2 template to run torchserve on ubuntu (#301)

* Adding a barebones ec2 template to run torchserve on ubuntu

* Adding VPC with Private subnet

* Address review comments #1

* Address review comments #2

* Rebase for #365 (#389)

* Updated README.md for #364

See #364 for a deep description of the problem

* updated README.md

* updated README.md

Co-authored-by: Francesco Saverio Zuppichini <[email protected]>

* Document the docker settings for running in production mode (#388)

* Docker Production Options #234

* Docker Production Options - Typo Fixes / Fit & Finish  #234

* Typos - Docker Docs #234

* grammar check  - docker/README.md

Co-authored-by: Aaqib <[email protected]>

* grammar check  - docker/README.md

Co-authored-by: Aaqib <[email protected]>

* grammar check  - docker/README.md

Co-authored-by: Aaqib <[email protected]>

Co-authored-by: Aaqib <[email protected]>

* Huggingface transformers integration test (#382)

* Add Huggingface transformers example integration tests

* Add transformers intallation

* Fix a missing bracket

* move transformers installation to chain installations

Co-authored-by: Geeta Chauhan <[email protected]>

* Updated docker README.md (#401)

* Update README.md

* Fixed cd command

* Made source cloning steps explicit!!

* Fixed online editing mistake

Co-authored-by: Geeta Chauhan <[email protected]>

* Fixed response status message for inferencing on model with zero workers (#305)

* Changes for issue_29

* Changed error Messages

* changed message for scalling request

* changed defaultVersion string

* Corrected error message

* changed the code for best practice

* resolved review comment for version

* updated management api doc as per new responses in worker scale api

Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Harsh Bafna <[email protected]>
Co-authored-by: dhanainme <[email protected]>
Co-authored-by: Shivam Shriwas <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* made model server port tests independent (#396)

* Added backend profiling and fixed benchmark's install dependency scripts (#402)

* added backend profiling
* updated install dependency scripts to make them jmeter version independent
* updated readme
* Fix benchmarks/README typo and JMeter versioning issue
* updated documentation

Co-authored-by: Aaqib Ansari <[email protected]>

* Add sphinx directory and conf.py (#274)

* Add sphinx folder that contains files for docs build

* Delete .DS_Store

* Delete .DS_Store

* Add conf.py in docs/

* documentation: fixed issues in installation and quick start in README.md

* documentation: Reverting conda related doc changes from PR#286

Fixes #297

* remove minus y (#311)

update pip install instructions

* Update to the latest version of sphinx

* Add recommonmark to extensions

Add recommonmark to extensions so sphinx project has markdown support.
Configuration details here: https://www.sphinx-doc.org/en/master/usage/markdown.html

* Update source suffix to use latest recommonmark syntax

Updating to the syntax found here: https://www.sphinx-doc.org/en/master/usage/markdown.html

Co-authored-by: Aaron Markham <[email protected]>

* Remove old recommonmark parser syntax

* Pin sphinx version to work with latest version of recommonmark

Co-authored-by: Aaron Markham <[email protected]>

* Pin version latest version recommonmark

Co-authored-by: Aaron Markham <[email protected]>

* Delete theme file generated during build

* de-duplicate install package of psutil (#329)

The Dockerfile of GPU install package psutil duplicated.

* Remove pytorch-sphinx-theme files

* Update to match latest staging_0_1_1

* Update docker directory to match staging_0_1_1 branch

* Delete .DS_Store

* Add sphinxcontrib.katex and torch

Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaqib Ansari <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>
Co-authored-by: Mario Cho <[email protected]>
Co-authored-by: mycpuorg <[email protected]>

* Image classifier example readme fixes (#405)

* Changed the model name in torch.jit.save to be consistent with docs

* Remove extra handler flag and fix the torchservr start

* fix the doc

* fix the doc

Co-authored-by: mycpuorg <[email protected]>

* Porting memory fix changes to TorchServe (#422)

Co-authored-by: dhanainme <[email protected]>

* Benchmark documentation fix (#425)

* Fix Benchmark Documentation

* Documentaion fixes

* .circleci: Add initial circleci configuration (#431)

Signed-off-by: Eli Uriegas <[email protected]>

* Torchscript support huggingface transfrmers (#421)

* Add support for torchscript

* Add support for torchscript

* Add key values for torchscript support

* Add Torchscript suport

* Add Torchscript support

* Add Torchscript support in loading the models

* keep the example to QA

* modify to reflect Torchscript example?

* Add torchscript support guides

* Add fintued BERT for QA

* Add QA artifacts

* Add seqenece classification artifacts

* Token classification artifacts

* Correct typos

* Add max length to tokenizer

* clean up

* enhance the doc

* change setup to seq classification

* Add detail to torchscript support

* set the model for QA

* Add torchscript bert seq classification example

* Add  bert seq classification example

* Modify the packaging model command

* Modify torchscript packaging command

* clean up

* clean up

* correct spacing between list items

Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Integration test huggingface transfromers torchscript support (#433)

* made model server port tests independent

* Add integration tests for Huggingface transfromers with new handler that support torchscript

Co-authored-by: harshbafna <[email protected]>
Co-authored-by: mycpuorg <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Apachebench benchmark - cleanup script / documentation

* Revert "Apachebench benchmark - cleanup script / documentation"

This reverts commit c8f8682.

* Documentation updates for clarity on Eager mode / Torch scripted models (#439)

* Model Zoo updates for Eager Mode / Torchscripted models

* Update examples/README.md

Co-authored-by: Aaqib <[email protected]>

Co-authored-by: Aaqib <[email protected]>

* Apachebench benchmarking - Script / Documentation updates (#438)

* Apachebench benchmarking script - Script / Documentation updates

* Stop TS after benchmark completion.

Co-authored-by: harshbafna <[email protected]>
Co-authored-by: dhaniram-kshirsagar <[email protected]>
Co-authored-by: dhaniram kshirsagar <[email protected]>
Co-authored-by: Aaqib <[email protected]>
Co-authored-by: mycpuorg <[email protected]>
Co-authored-by: Harsh Bafna <[email protected]>
Co-authored-by: mjpsl <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Vamshidhar Dantu <[email protected]>
Co-authored-by: Vamshidhar Dantu <[email protected]>
Co-authored-by: Brad Heintz <[email protected]>
Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>
Co-authored-by: alexwong <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: dkasar-aws <[email protected]>
Co-authored-by: Shivam Shriwas <[email protected]>
Co-authored-by: Hamid Shojanazeri <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: shivamshriwas <[email protected]>
Co-authored-by: Takayoshi Nishida <[email protected]>
Co-authored-by: alvations <[email protected]>
Co-authored-by: Tzu-Wei Huang <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Francesco Saverio Zuppichini <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Jessica Lin <[email protected]>
Co-authored-by: Mario Cho <[email protected]>
Co-authored-by: Eli Uriegas <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
maaquib added a commit that referenced this pull request Jun 9, 2020
* fixed resnet152 custom handler to work with GPU

* issue template

* Moving issue templates to root folder per pull request template

* adding feature request template

* Minor changes related to sections and names.

* Update feature_template.md

* Update bug_template.md

* doc template

* minor changes

* Minor changes to upload custom model or handler

* issue templates (#273)

* issue template

* Moving issue templates to root folder per pull request template

* adding feature request template

* Minor changes related to sections and names.

* Update feature_template.md

* Update bug_template.md

* doc template

* minor changes

* Minor changes to upload custom model or handler

Co-authored-by: mycpuorg <[email protected]>

* Fixes #205 - Docker install not finding GPUs (#262)

Invokes docker with  --runtime=nvidia for GPU
    Added the option to specify the GPU / specific GPU ids in start.sh
    Fixed the documentation for start.sh script
    Fixed the JDK version in Dockerfile.gpu

Co-authored-by: Aaqib <[email protected]>

* Fix pylint issue (#163)

* pylint fixes and code cleanup

* UT fixes, pylint updates per latest version and updated sanity script to run pylint

* fixed pylint and wrapt installation issue

* freeze pylint version to 2.4.4 due to bug in version 2.5

* removed duplicate statement to install pylint

Co-authored-by: Aaqib <[email protected]>

* Updated exception handling for worker threads shutdown. (#172)

* updated code to now log errors only in case when exception is not genreated by unregister api call

* java formatting

* fixed illigalstateexception

* removed the unrequired handling for illegal monitor state exception

Co-authored-by: dhaniram kshirsagar <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* changes for Homebrew  issue 98 (#126)

* changes for Homebrew  issue 98

* adding changes for path in benchmark.py

* CI Build corrected

* benchmark changes

corrected path

* changed PATH for homebrew

* Update Readme

Made changes for homebrew installation and instruction for GPU based machine instruction for install_dependencies.sh

* Updated Readme.md

removed unnecessory  lines

* corrected indentation

corrected indentation for  new lines added

* Update Readme.md fil

added separate commands for CPU and GPU based

Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: harshbafna <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Harsh Bafna <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* added changes to clean model loaded through URL from model store on unregisted

* added UTs

* Moving the plugins SDK to org.pytorch. Added plugins example. (#261)

* Moved plugins SDK to org.pytorch

* Updating the repo for publishing plugins SDK

* Enabled bintray maven repo temporarily

* Reset tests

* Added GPG plugin to pom file to sign the plugins SDK

* Update the SDK version

* Removed bintray repo

* Modify the buildspec to not deploy the SDK

Co-authored-by: Vamshidhar Dantu <[email protected]>
Co-authored-by: Aaqib <[email protected]>
Co-authored-by: mycpuorg <[email protected]>

* Added model zoo (#299)

Co-authored-by: mycpuorg <[email protected]>
Co-authored-by: Brad Heintz <[email protected]>

* Removed snapshot dependency on model (#152)

* Snapshot is now independent of model

* fixed checkstyle issue

* fixed formatting

* doc review: configuration and custom_service

* addressed review comments

* fixed incorrect merge

* fixed review comments and corrected the error messages

* adding remaining fixes for string constants

Co-authored-by: harshbafna <[email protected]>
Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>
Co-authored-by: Harsh Bafna <[email protected]>

* Regression Tests Suite - AWS Code Build Infra - Initial Commit for #57 (#285)

* Regression Tests Suite - AWS Code Build Infra - Initial Commit for #57

* Script to clone Pytorch Repo & Run Post Man Scripts
* Basic Tests for Inference & Managment APIs
* AWS Code Build buildspec.yml

* Refactor postman tests, add empty postman collection for todos

* Add a readme for adding tests

* Add more management tests

* Fix typo

* Snapshot Regression tests - #57

* Fix typo

* Comment out snapshot test for now

* Weird typos are showing up, try saving everything

* Need to manually save every request in postman...

* When reregister don't use s3

* More workarounds for rereg

* Fix test messages

* Remove test for empty reqs

* Change get to post

* Fix pytest for subprocess - #57

* Enable pytest - #57

* Inference API Testcases / Rename Management Testcases  #57

* Wipe Model MARs in Modelstore between tests #54

* Fix Inference Endpoint Execution #54

* Fix file paths - #57

* Fix Initial Workers #57

* Fix Inference Assertions #57

* Fix DenseNet Inference #57

* Fix Densenet Register #54

* Redirect Logs to File #54

* Fix Test Execution Log #54

* README, Build Badge #57

* Format ReadMe #57

* Update README - Add Instruction to add new tests / Run tests locally

* Update README

* Update branch for build Codebuild Badge

Co-authored-by: alexwong <[email protected]>
Co-authored-by: Aaqib <[email protected]>
Co-authored-by: mycpuorg <[email protected]>
Co-authored-by: Ubuntu <[email protected]>

* Issue 312 install from src (#322)

* documentation: fixed issues in installation and quick start in README.md

* documentation: Reverting conda related doc changes from PR#286

Fixes #297

* remove minus y (#311)

update pip install instructions

* Update instructions for installation from source

This commit aims to address Issue #312

Tested on a fresh EC2 P3.2 instance:
* Followed instructions under
https://github.com/pytorch/serve#install-torchserve-for-development
* Started the server manually from command line
* Ran a ping query

Received a response from server with "Healthy" status

Signed-off-by: Manoj Rao <[email protected]>

* add separate files for installation

one each for linux and macos

* update README and remove relative path from script

* Install script - Dont uninstall if not previously installed - #132

* explicitly build the source code with gradle prior to install

* remove upgrade instructions in favor of clean reinstall from source

* update instructions to contain the correct script name

Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaqib Ansari <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>
Co-authored-by: Manoj Rao <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>

* fix for macos script (#338)

* Move Issue Template to .github dir

* Updated sanity script to use install_from_src_ubuntu script (#350)

* updated sanity script to use install_from_src_ubuntu script

* added java version check

* fixed issue with if statement

* fixed variable name

* fixed repeated cd command

* updated artifacts

* removed artifacts from buildspec

* Corrected HTTP response code for Model version does not exists exception (#282)

* issue 196 patch

* Added openapi missing responses. - incorporated review comments

* updated method signature after resolving merge conflicts

Co-authored-by: Harsh Bafna <[email protected]>
Co-authored-by: Aaqib <[email protected]>
Co-authored-by: Shivam Shriwas <[email protected]>
Co-authored-by: harshbafna <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Corrected paths in command and added notes (#354)

Co-authored-by: dhanainme <[email protected]>

* Refactored UT for better reporting (#159)

* refactored UTs

* upgraded testng version and updated testcase names

* added testng.xml to handle sequential execution of test classes

* added waveglow example

* Updated readme for waveglow example

* fixed failing UTs after merging master

* removed unintentionally pushed waveglow example

* fixed failing test case

* moved set-default to utils and added couple of more add cases for default version

* java formatting

* corrected set default negative test case

* fixed java formatting

Co-authored-by: Aaqib <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Huggingface Transformers generalization (#302)

* Scripts to seriliaze a Bert model using jit.trace and a handler for Bert model

* modified to include mapping of predicitons and decoding received text

* move back the original text

* moved the index_to_name to bert folder

* Transformers/ Bert serving with Torchserve readme

* deleting an unused dictionary

* just cleaning some of the commented out parts

* cleaning up

* Cleaning up

* uncommenting the logging and cleaning  the comments

* Download and save the checkpoints, config file and vocab of  the BERT from Transformers for demonstration purposes

* mapping of predicitons to labels

* a custom handler for transformers.

* README.

* sample of input text.

* modifying the descriptions

* modifying the descriptions

* adding comments

* adding commnets

* Fromatting modifications

* modified comments

* added the citations

* adding hugginface as the type of transformer model we are referring to

* renaming the folder to huggingface_transformers

* Removing the bert folder

* simple modification

* modifed the script to use a setup_config.json for loading and saving models

* added the section for transfomer_handler_generalized

* the generalized handler

* set_up config intialization

* edited for transfomer generalization

* modified to use just input_ids as inputs for models

* modified for question answering

* modified to include the number of labels

* corrected typos

* modifed variable names

* edited for clarity

* modifed variable names, simplified the initlialize in the handler class, removed the warnning from QA case

* modifed values for QA

* modifed the variable names

* added comments

* modified to add clarity

* add comment in preprocessing of QA

* added warning for the case mode is not selected properly and combined constructor and initlaize fucntion

* clean up

* clean up

* combined constructor and initliaze func

* combined constructor and initliaze func

* Separated the constructor and initliaze func

* Modified to remove the classification part and update with generalized version

* resolved the coflict and separated the costructor and initliaze func

* removed the handler for classification case as the generalized version handles the case

Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: mycpuorg <[email protected]>

* added waveglow example (#258)

* added waveglow example

* Updated readme for waveglow example

* enhanced example to remove runtime checkpoint download

* updated handler to return the audio file content instead of dumping it to tmp dir

* minor change to handler

* added pipefail

* added python snippet to run inference

* updated readme with know issue with the example

Co-authored-by: Aaqib <[email protected]>

* made build suite compatible with latest gradle (#348)

Co-authored-by: dhanainme <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* moved env section after context

* Deleting redundant ISSUE_TEMPLATE at root dir

* Fix typo in scripts/install_from_src_macos

* Corrected  OpenApi description  (#328)

* Correct query parameter name (Registration Api)in OpenApi description

* removed non existing path parameter

Co-authored-by: Aaqib <[email protected]>

* Regression suite fixes and added test cases (#344)

* added test model zoo test cases
updated regression script

* Updated Readme

* update -y flag for install

* Updated docker command as per new imagename and tag generated in stagging branch

* Fix sudo usage in regression test script

* - addressed review comments
- reverted changes related to sudo in regression script.

* added https test collection

* - updated failed test cases on staging branch
- added environment and updated newman cmd  with --insecure for https

* updated script

* updated pytest

* update curl check for serve up

* Revert "update curl check for serve up"

This reverts commit 0047eae.

* Revert "update curl check for serve up"

This reverts commit 0047eae.

- update model zoo
- added resnet test case

* added workers to resnet model

* Update readme to run test suite on desired branch

* enhanced inference test cases to driven by datafile

* Updated readme

Co-authored-by: Dhanasekar Karuppasamy <[email protected]>
Co-authored-by: Aaqib <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Fix typo in mnist readme (#330)

Co-authored-by: Aaqib <[email protected]>

* Fixed typo in ab installation. (#327)

* documentation: fixed issues in installation and quick start in README.md

* documentation: Reverting conda related doc changes from PR#286

Fixes #297

* remove minus y (#311)

update pip install instructions

* Fixed typo in ab installation.

Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaqib Ansari <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>

* Setup Nightly Build for TorchServe (#371)

* Add Branch Flag for Nightly CodeBuild #57

* Update ENV Variable for Nightly Build Name

* Nigtly Build - Fix FileName for Build Artifacts

* Update README for accessing test logs

* Whitspace changes

* Build new images for build - README Instruction

* Fix for intermittent gradle build failure (#352)

* Fix for intermittent null pointer exception

* Fixed comments

Co-authored-by: dhanainme <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* documentation: Remove redundant curl options in example (#336)

* documentation: fixed issues in installation and quick start in README.md

* documentation: Reverting conda related doc changes from PR#286

Fixes #297

* remove minus y (#311)

update pip install instructions

* fix-doc

* fix remaining "-X POST"s

Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaqib Ansari <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>

* Corrected port number validation range (#304)

* Corrected port number validation range

* added UT for issue 291

* added ut dependency file formatting

* correct param in config_test_env.properties

* removed incorrect flags from test case config files

* added negative test cases

* removed properties file and changes in test case

Co-authored-by: Harsh Bafna <[email protected]>

* Fixes version number reporting for TorchServe (#360)

* Correct version and added vesion.txt for getting version info

* fixed description test cases and build issue

* Added snapshot test fixes
and resolved relative path issue for version.txt

* python formatting

* added version.txt in MANIFEST.in

* addressed review comments

* added version.txt for model archiver as well
- addressed review comments

* -fixed python UT
-added ts python UT run in sanity script

* removed blank lines from snapshot files

Co-authored-by: Aaqib <[email protected]>
Co-authored-by: Harsh Bafna <[email protected]>

* fixes installation script exits without error if no java present on machine (#376)

* fixes installation script exits without error if no java present on machine

* added changes for macos script

Co-authored-by: Aaqib <[email protected]>

* Dockerfile enhancements (#308)

* dockerfile now points to latest torchserve on pypi and added separate files for dev

* Minor changes

* Made build and run commands consistent

* Fixing formatting

* Consistency issue

* Enhanced Dockerfile for binary per #178 and other misc.

* Fixed gpu related issues in start script

* Corrected the heading font

* Added mar creation steps from container

* added buildkit support for dev docker files

* Fix for buildkit

* Restoring psutil and future as absence leads intermittent hangged build

* Fixed review comments for doc

* Fixed typo

Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* Updated configuration.md for missing config properties (#337)

* Updated configuration.md for missing parameters

* changes TS_MODEL_SNAPSHOT to MODEL_SNAPSHOT
- removes ambiguity
- follows naming convention

* addressed review comments.

* removed debug property

* updated configuration.md

Co-authored-by: Harsh Bafna <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* Adding missing document links to README (#379)

* Maven tests should output only errors (#326)

Co-authored-by: Vamshidhar Dantu <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* fixed code for build failure in python virtual environment (#341)

* fixed code for build failure in python virtual environment

* added exception missed on resolving conflicts

Co-authored-by: Aaqib <[email protected]>
Co-authored-by: Vamshidhar Dantu <[email protected]>

* updated object detector handler to use only default cuda device for torchvision version less then 0.6 (#265)

* updated object detector handler to use default cuda device for torchvision version 0.5.0

* updated version check logic

* Updated documentation for torchvision version dependency

* enhanced sanity script to validate all default handlers

* fixed sanity script

* added torchtext install step in sanity script

* fixed issue related to inference after restart with snapshot.

* moved torchtext to codebuild docker files

* added more log statements

* enhanced run inference function

* fixed register model function

* added unregister model function

* incorporated code review comments

* added back inference call for snapshot test

* fixed if block

Co-authored-by: Aaqib <[email protected]>

* Adding a barebones ec2 template to run torchserve on ubuntu (#301)

* Adding a barebones ec2 template to run torchserve on ubuntu

* Adding VPC with Private subnet

* Address review comments #1

* Address review comments #2

* Rebase for #365 (#389)

* Updated README.md for #364

See #364 for a deep description of the problem

* updated README.md

* updated README.md

Co-authored-by: Francesco Saverio Zuppichini <[email protected]>

* Document the docker settings for running in production mode (#388)

* Docker Production Options #234

* Docker Production Options - Typo Fixes / Fit & Finish  #234

* Typos - Docker Docs #234

* grammar check  - docker/README.md

Co-authored-by: Aaqib <[email protected]>

* grammar check  - docker/README.md

Co-authored-by: Aaqib <[email protected]>

* grammar check  - docker/README.md

Co-authored-by: Aaqib <[email protected]>

Co-authored-by: Aaqib <[email protected]>

* Huggingface transformers integration test (#382)

* Add Huggingface transformers example integration tests

* Add transformers intallation

* Fix a missing bracket

* move transformers installation to chain installations

Co-authored-by: Geeta Chauhan <[email protected]>

* Updated docker README.md (#401)

* Update README.md

* Fixed cd command

* Made source cloning steps explicit!!

* Fixed online editing mistake

Co-authored-by: Geeta Chauhan <[email protected]>

* Fixed response status message for inferencing on model with zero workers (#305)

* Changes for issue_29

* Changed error Messages

* changed message for scalling request

* changed defaultVersion string

* Corrected error message

* changed the code for best practice

* resolved review comment for version

* updated management api doc as per new responses in worker scale api

Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Harsh Bafna <[email protected]>
Co-authored-by: dhanainme <[email protected]>
Co-authored-by: Shivam Shriwas <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* made model server port tests independent (#396)

* Added backend profiling and fixed benchmark's install dependency scripts (#402)

* added backend profiling
* updated install dependency scripts to make them jmeter version independent
* updated readme
* Fix benchmarks/README typo and JMeter versioning issue
* updated documentation

Co-authored-by: Aaqib Ansari <[email protected]>

* Add sphinx directory and conf.py (#274)

* Add sphinx folder that contains files for docs build

* Delete .DS_Store

* Delete .DS_Store

* Add conf.py in docs/

* documentation: fixed issues in installation and quick start in README.md

* documentation: Reverting conda related doc changes from PR#286

Fixes #297

* remove minus y (#311)

update pip install instructions

* Update to the latest version of sphinx

* Add recommonmark to extensions

Add recommonmark to extensions so sphinx project has markdown support.
Configuration details here: https://www.sphinx-doc.org/en/master/usage/markdown.html

* Update source suffix to use latest recommonmark syntax

Updating to the syntax found here: https://www.sphinx-doc.org/en/master/usage/markdown.html

Co-authored-by: Aaron Markham <[email protected]>

* Remove old recommonmark parser syntax

* Pin sphinx version to work with latest version of recommonmark

Co-authored-by: Aaron Markham <[email protected]>

* Pin version latest version recommonmark

Co-authored-by: Aaron Markham <[email protected]>

* Delete theme file generated during build

* de-duplicate install package of psutil (#329)

The Dockerfile of GPU install package psutil duplicated.

* Remove pytorch-sphinx-theme files

* Update to match latest staging_0_1_1

* Update docker directory to match staging_0_1_1 branch

* Delete .DS_Store

* Add sphinxcontrib.katex and torch

Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaqib Ansari <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>
Co-authored-by: Mario Cho <[email protected]>
Co-authored-by: mycpuorg <[email protected]>

* Image classifier example readme fixes (#405)

* Changed the model name in torch.jit.save to be consistent with docs

* Remove extra handler flag and fix the torchservr start

* fix the doc

* fix the doc

Co-authored-by: mycpuorg <[email protected]>

* Porting memory fix changes to TorchServe (#422)

Co-authored-by: dhanainme <[email protected]>

* Benchmark documentation fix (#425)

* Fix Benchmark Documentation

* Documentaion fixes

* .circleci: Add initial circleci configuration (#431)

Signed-off-by: Eli Uriegas <[email protected]>

* Torchscript support huggingface transfrmers (#421)

* Add support for torchscript

* Add support for torchscript

* Add key values for torchscript support

* Add Torchscript suport

* Add Torchscript support

* Add Torchscript support in loading the models

* keep the example to QA

* modify to reflect Torchscript example?

* Add torchscript support guides

* Add fintued BERT for QA

* Add QA artifacts

* Add seqenece classification artifacts

* Token classification artifacts

* Correct typos

* Add max length to tokenizer

* clean up

* enhance the doc

* change setup to seq classification

* Add detail to torchscript support

* set the model for QA

* Add torchscript bert seq classification example

* Add  bert seq classification example

* Modify the packaging model command

* Modify torchscript packaging command

* clean up

* clean up

* correct spacing between list items

Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Integration test huggingface transfromers torchscript support (#433)

* made model server port tests independent

* Add integration tests for Huggingface transfromers with new handler that support torchscript

Co-authored-by: harshbafna <[email protected]>
Co-authored-by: mycpuorg <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Apachebench benchmark - cleanup script / documentation

* Revert "Apachebench benchmark - cleanup script / documentation"

This reverts commit c8f8682.

* Documentation updates for clarity on Eager mode / Torch scripted models (#439)

* Model Zoo updates for Eager Mode / Torchscripted models

* Update examples/README.md

Co-authored-by: Aaqib <[email protected]>

Co-authored-by: Aaqib <[email protected]>

* Apachebench benchmarking - Script / Documentation updates (#438)

* Apachebench benchmarking script - Script / Documentation updates

* Stop TS after benchmark completion.

Co-authored-by: harshbafna <[email protected]>
Co-authored-by: dhaniram-kshirsagar <[email protected]>
Co-authored-by: dhaniram kshirsagar <[email protected]>
Co-authored-by: Aaqib <[email protected]>
Co-authored-by: mycpuorg <[email protected]>
Co-authored-by: Harsh Bafna <[email protected]>
Co-authored-by: mjpsl <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Vamshidhar Dantu <[email protected]>
Co-authored-by: Vamshidhar Dantu <[email protected]>
Co-authored-by: Brad Heintz <[email protected]>
Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>
Co-authored-by: alexwong <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: dkasar-aws <[email protected]>
Co-authored-by: Shivam Shriwas <[email protected]>
Co-authored-by: Hamid Shojanazeri <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: shivamshriwas <[email protected]>
Co-authored-by: Takayoshi Nishida <[email protected]>
Co-authored-by: alvations <[email protected]>
Co-authored-by: Tzu-Wei Huang <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Francesco Saverio Zuppichini <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Jessica Lin <[email protected]>
Co-authored-by: Mario Cho <[email protected]>
Co-authored-by: Eli Uriegas <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
@harshbafna harshbafna deleted the issue_204 branch June 15, 2020 04:05
dhanainme added a commit that referenced this pull request Jun 25, 2020
* fixed resnet152 custom handler to work with GPU

* issue template

* Moving issue templates to root folder per pull request template

* adding feature request template

* Minor changes related to sections and names.

* Update feature_template.md

* Update bug_template.md

* doc template

* minor changes

* Minor changes to upload custom model or handler

* issue templates (#273)

* issue template

* Moving issue templates to root folder per pull request template

* adding feature request template

* Minor changes related to sections and names.

* Update feature_template.md

* Update bug_template.md

* doc template

* minor changes

* Minor changes to upload custom model or handler

Co-authored-by: mycpuorg <[email protected]>

* Fixes #205 - Docker install not finding GPUs (#262)

Invokes docker with  --runtime=nvidia for GPU
    Added the option to specify the GPU / specific GPU ids in start.sh
    Fixed the documentation for start.sh script
    Fixed the JDK version in Dockerfile.gpu

Co-authored-by: Aaqib <[email protected]>

* Fix pylint issue (#163)

* pylint fixes and code cleanup

* UT fixes, pylint updates per latest version and updated sanity script to run pylint

* fixed pylint and wrapt installation issue

* freeze pylint version to 2.4.4 due to bug in version 2.5

* removed duplicate statement to install pylint

Co-authored-by: Aaqib <[email protected]>

* Updated exception handling for worker threads shutdown. (#172)

* updated code to now log errors only in case when exception is not genreated by unregister api call

* java formatting

* fixed illigalstateexception

* removed the unrequired handling for illegal monitor state exception

Co-authored-by: dhaniram kshirsagar <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* changes for Homebrew  issue 98 (#126)

* changes for Homebrew  issue 98

* adding changes for path in benchmark.py

* CI Build corrected

* benchmark changes

corrected path

* changed PATH for homebrew

* Update Readme

Made changes for homebrew installation and instruction for GPU based machine instruction for install_dependencies.sh

* Updated Readme.md

removed unnecessory  lines

* corrected indentation

corrected indentation for  new lines added

* Update Readme.md fil

added separate commands for CPU and GPU based

Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: harshbafna <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Harsh Bafna <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* added changes to clean model loaded through URL from model store on unregisted

* added UTs

* Moving the plugins SDK to org.pytorch. Added plugins example. (#261)

* Moved plugins SDK to org.pytorch

* Updating the repo for publishing plugins SDK

* Enabled bintray maven repo temporarily

* Reset tests

* Added GPG plugin to pom file to sign the plugins SDK

* Update the SDK version

* Removed bintray repo

* Modify the buildspec to not deploy the SDK

Co-authored-by: Vamshidhar Dantu <[email protected]>
Co-authored-by: Aaqib <[email protected]>
Co-authored-by: mycpuorg <[email protected]>

* Added model zoo (#299)

Co-authored-by: mycpuorg <[email protected]>
Co-authored-by: Brad Heintz <[email protected]>

* Removed snapshot dependency on model (#152)

* Snapshot is now independent of model

* fixed checkstyle issue

* fixed formatting

* doc review: configuration and custom_service

* addressed review comments

* fixed incorrect merge

* fixed review comments and corrected the error messages

* adding remaining fixes for string constants

Co-authored-by: harshbafna <[email protected]>
Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>
Co-authored-by: Harsh Bafna <[email protected]>

* Regression Tests Suite - AWS Code Build Infra - Initial Commit for #57 (#285)

* Regression Tests Suite - AWS Code Build Infra - Initial Commit for #57

* Script to clone Pytorch Repo & Run Post Man Scripts
* Basic Tests for Inference & Managment APIs
* AWS Code Build buildspec.yml

* Refactor postman tests, add empty postman collection for todos

* Add a readme for adding tests

* Add more management tests

* Fix typo

* Snapshot Regression tests - #57

* Fix typo

* Comment out snapshot test for now

* Weird typos are showing up, try saving everything

* Need to manually save every request in postman...

* When reregister don't use s3

* More workarounds for rereg

* Fix test messages

* Remove test for empty reqs

* Change get to post

* Fix pytest for subprocess - #57

* Enable pytest - #57

* Inference API Testcases / Rename Management Testcases  #57

* Wipe Model MARs in Modelstore between tests #54

* Fix Inference Endpoint Execution #54

* Fix file paths - #57

* Fix Initial Workers #57

* Fix Inference Assertions #57

* Fix DenseNet Inference #57

* Fix Densenet Register #54

* Redirect Logs to File #54

* Fix Test Execution Log #54

* README, Build Badge #57

* Format ReadMe #57

* Update README - Add Instruction to add new tests / Run tests locally

* Update README

* Update branch for build Codebuild Badge

Co-authored-by: alexwong <[email protected]>
Co-authored-by: Aaqib <[email protected]>
Co-authored-by: mycpuorg <[email protected]>
Co-authored-by: Ubuntu <[email protected]>

* Issue 312 install from src (#322)

* documentation: fixed issues in installation and quick start in README.md

* documentation: Reverting conda related doc changes from PR#286

Fixes #297

* remove minus y (#311)

update pip install instructions

* Update instructions for installation from source

This commit aims to address Issue #312

Tested on a fresh EC2 P3.2 instance:
* Followed instructions under
https://github.com/pytorch/serve#install-torchserve-for-development
* Started the server manually from command line
* Ran a ping query

Received a response from server with "Healthy" status

Signed-off-by: Manoj Rao <[email protected]>

* add separate files for installation

one each for linux and macos

* update README and remove relative path from script

* Install script - Dont uninstall if not previously installed - #132

* explicitly build the source code with gradle prior to install

* remove upgrade instructions in favor of clean reinstall from source

* update instructions to contain the correct script name

Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaqib Ansari <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>
Co-authored-by: Manoj Rao <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>

* fix for macos script (#338)

* Move Issue Template to .github dir

* Updated sanity script to use install_from_src_ubuntu script (#350)

* updated sanity script to use install_from_src_ubuntu script

* added java version check

* fixed issue with if statement

* fixed variable name

* fixed repeated cd command

* updated artifacts

* removed artifacts from buildspec

* Corrected HTTP response code for Model version does not exists exception (#282)

* issue 196 patch

* Added openapi missing responses. - incorporated review comments

* updated method signature after resolving merge conflicts

Co-authored-by: Harsh Bafna <[email protected]>
Co-authored-by: Aaqib <[email protected]>
Co-authored-by: Shivam Shriwas <[email protected]>
Co-authored-by: harshbafna <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Corrected paths in command and added notes (#354)

Co-authored-by: dhanainme <[email protected]>

* Refactored UT for better reporting (#159)

* refactored UTs

* upgraded testng version and updated testcase names

* added testng.xml to handle sequential execution of test classes

* added waveglow example

* Updated readme for waveglow example

* fixed failing UTs after merging master

* removed unintentionally pushed waveglow example

* fixed failing test case

* moved set-default to utils and added couple of more add cases for default version

* java formatting

* corrected set default negative test case

* fixed java formatting

Co-authored-by: Aaqib <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Huggingface Transformers generalization (#302)

* Scripts to seriliaze a Bert model using jit.trace and a handler for Bert model

* modified to include mapping of predicitons and decoding received text

* move back the original text

* moved the index_to_name to bert folder

* Transformers/ Bert serving with Torchserve readme

* deleting an unused dictionary

* just cleaning some of the commented out parts

* cleaning up

* Cleaning up

* uncommenting the logging and cleaning  the comments

* Download and save the checkpoints, config file and vocab of  the BERT from Transformers for demonstration purposes

* mapping of predicitons to labels

* a custom handler for transformers.

* README.

* sample of input text.

* modifying the descriptions

* modifying the descriptions

* adding comments

* adding commnets

* Fromatting modifications

* modified comments

* added the citations

* adding hugginface as the type of transformer model we are referring to

* renaming the folder to huggingface_transformers

* Removing the bert folder

* simple modification

* modifed the script to use a setup_config.json for loading and saving models

* added the section for transfomer_handler_generalized

* the generalized handler

* set_up config intialization

* edited for transfomer generalization

* modified to use just input_ids as inputs for models

* modified for question answering

* modified to include the number of labels

* corrected typos

* modifed variable names

* edited for clarity

* modifed variable names, simplified the initlialize in the handler class, removed the warnning from QA case

* modifed values for QA

* modifed the variable names

* added comments

* modified to add clarity

* add comment in preprocessing of QA

* added warning for the case mode is not selected properly and combined constructor and initlaize fucntion

* clean up

* clean up

* combined constructor and initliaze func

* combined constructor and initliaze func

* Separated the constructor and initliaze func

* Modified to remove the classification part and update with generalized version

* resolved the coflict and separated the costructor and initliaze func

* removed the handler for classification case as the generalized version handles the case

Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: mycpuorg <[email protected]>

* added waveglow example (#258)

* added waveglow example

* Updated readme for waveglow example

* enhanced example to remove runtime checkpoint download

* updated handler to return the audio file content instead of dumping it to tmp dir

* minor change to handler

* added pipefail

* added python snippet to run inference

* updated readme with know issue with the example

Co-authored-by: Aaqib <[email protected]>

* made build suite compatible with latest gradle (#348)

Co-authored-by: dhanainme <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* moved env section after context

* Deleting redundant ISSUE_TEMPLATE at root dir

* Fix typo in scripts/install_from_src_macos

* Corrected  OpenApi description  (#328)

* Correct query parameter name (Registration Api)in OpenApi description

* removed non existing path parameter

Co-authored-by: Aaqib <[email protected]>

* Regression suite fixes and added test cases (#344)

* added test model zoo test cases
updated regression script

* Updated Readme

* update -y flag for install

* Updated docker command as per new imagename and tag generated in stagging branch

* Fix sudo usage in regression test script

* - addressed review comments
- reverted changes related to sudo in regression script.

* added https test collection

* - updated failed test cases on staging branch
- added environment and updated newman cmd  with --insecure for https

* updated script

* updated pytest

* update curl check for serve up

* Revert "update curl check for serve up"

This reverts commit 0047eae.

* Revert "update curl check for serve up"

This reverts commit 0047eae.

- update model zoo
- added resnet test case

* added workers to resnet model

* Update readme to run test suite on desired branch

* enhanced inference test cases to driven by datafile

* Updated readme

Co-authored-by: Dhanasekar Karuppasamy <[email protected]>
Co-authored-by: Aaqib <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Fix typo in mnist readme (#330)

Co-authored-by: Aaqib <[email protected]>

* Fixed typo in ab installation. (#327)

* documentation: fixed issues in installation and quick start in README.md

* documentation: Reverting conda related doc changes from PR#286

Fixes #297

* remove minus y (#311)

update pip install instructions

* Fixed typo in ab installation.

Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaqib Ansari <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>

* Setup Nightly Build for TorchServe (#371)

* Add Branch Flag for Nightly CodeBuild #57

* Update ENV Variable for Nightly Build Name

* Nigtly Build - Fix FileName for Build Artifacts

* Update README for accessing test logs

* Whitspace changes

* Build new images for build - README Instruction

* Fix for intermittent gradle build failure (#352)

* Fix for intermittent null pointer exception

* Fixed comments

Co-authored-by: dhanainme <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* documentation: Remove redundant curl options in example (#336)

* documentation: fixed issues in installation and quick start in README.md

* documentation: Reverting conda related doc changes from PR#286

Fixes #297

* remove minus y (#311)

update pip install instructions

* fix-doc

* fix remaining "-X POST"s

Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaqib Ansari <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>

* Corrected port number validation range (#304)

* Corrected port number validation range

* added UT for issue 291

* added ut dependency file formatting

* correct param in config_test_env.properties

* removed incorrect flags from test case config files

* added negative test cases

* removed properties file and changes in test case

Co-authored-by: Harsh Bafna <[email protected]>

* Fixes version number reporting for TorchServe (#360)

* Correct version and added vesion.txt for getting version info

* fixed description test cases and build issue

* Added snapshot test fixes
and resolved relative path issue for version.txt

* python formatting

* added version.txt in MANIFEST.in

* addressed review comments

* added version.txt for model archiver as well
- addressed review comments

* -fixed python UT
-added ts python UT run in sanity script

* removed blank lines from snapshot files

Co-authored-by: Aaqib <[email protected]>
Co-authored-by: Harsh Bafna <[email protected]>

* fixes installation script exits without error if no java present on machine (#376)

* fixes installation script exits without error if no java present on machine

* added changes for macos script

Co-authored-by: Aaqib <[email protected]>

* Dockerfile enhancements (#308)

* dockerfile now points to latest torchserve on pypi and added separate files for dev

* Minor changes

* Made build and run commands consistent

* Fixing formatting

* Consistency issue

* Enhanced Dockerfile for binary per #178 and other misc.

* Fixed gpu related issues in start script

* Corrected the heading font

* Added mar creation steps from container

* added buildkit support for dev docker files

* Fix for buildkit

* Restoring psutil and future as absence leads intermittent hangged build

* Fixed review comments for doc

* Fixed typo

Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* Updated configuration.md for missing config properties (#337)

* Updated configuration.md for missing parameters

* changes TS_MODEL_SNAPSHOT to MODEL_SNAPSHOT
- removes ambiguity
- follows naming convention

* addressed review comments.

* removed debug property

* updated configuration.md

Co-authored-by: Harsh Bafna <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* Adding missing document links to README (#379)

* Maven tests should output only errors (#326)

Co-authored-by: Vamshidhar Dantu <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* fixed code for build failure in python virtual environment (#341)

* fixed code for build failure in python virtual environment

* added exception missed on resolving conflicts

Co-authored-by: Aaqib <[email protected]>
Co-authored-by: Vamshidhar Dantu <[email protected]>

* updated object detector handler to use only default cuda device for torchvision version less then 0.6 (#265)

* updated object detector handler to use default cuda device for torchvision version 0.5.0

* updated version check logic

* Updated documentation for torchvision version dependency

* enhanced sanity script to validate all default handlers

* fixed sanity script

* added torchtext install step in sanity script

* fixed issue related to inference after restart with snapshot.

* moved torchtext to codebuild docker files

* added more log statements

* enhanced run inference function

* fixed register model function

* added unregister model function

* incorporated code review comments

* added back inference call for snapshot test

* fixed if block

Co-authored-by: Aaqib <[email protected]>

* Adding a barebones ec2 template to run torchserve on ubuntu (#301)

* Adding a barebones ec2 template to run torchserve on ubuntu

* Adding VPC with Private subnet

* Address review comments #1

* Address review comments #2

* Rebase for #365 (#389)

* Updated README.md for #364

See #364 for a deep description of the problem

* updated README.md

* updated README.md

Co-authored-by: Francesco Saverio Zuppichini <[email protected]>

* Document the docker settings for running in production mode (#388)

* Docker Production Options #234

* Docker Production Options - Typo Fixes / Fit & Finish  #234

* Typos - Docker Docs #234

* grammar check  - docker/README.md

Co-authored-by: Aaqib <[email protected]>

* grammar check  - docker/README.md

Co-authored-by: Aaqib <[email protected]>

* grammar check  - docker/README.md

Co-authored-by: Aaqib <[email protected]>

Co-authored-by: Aaqib <[email protected]>

* Huggingface transformers integration test (#382)

* Add Huggingface transformers example integration tests

* Add transformers intallation

* Fix a missing bracket

* move transformers installation to chain installations

Co-authored-by: Geeta Chauhan <[email protected]>

* Updated docker README.md (#401)

* Update README.md

* Fixed cd command

* Made source cloning steps explicit!!

* Fixed online editing mistake

Co-authored-by: Geeta Chauhan <[email protected]>

* Fixed response status message for inferencing on model with zero workers (#305)

* Changes for issue_29

* Changed error Messages

* changed message for scalling request

* changed defaultVersion string

* Corrected error message

* changed the code for best practice

* resolved review comment for version

* updated management api doc as per new responses in worker scale api

Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Harsh Bafna <[email protected]>
Co-authored-by: dhanainme <[email protected]>
Co-authored-by: Shivam Shriwas <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* made model server port tests independent (#396)

* Added backend profiling and fixed benchmark's install dependency scripts (#402)

* added backend profiling
* updated install dependency scripts to make them jmeter version independent
* updated readme
* Fix benchmarks/README typo and JMeter versioning issue
* updated documentation

Co-authored-by: Aaqib Ansari <[email protected]>

* Add sphinx directory and conf.py (#274)

* Add sphinx folder that contains files for docs build

* Delete .DS_Store

* Delete .DS_Store

* Add conf.py in docs/

* documentation: fixed issues in installation and quick start in README.md

* documentation: Reverting conda related doc changes from PR#286

Fixes #297

* remove minus y (#311)

update pip install instructions

* Update to the latest version of sphinx

* Add recommonmark to extensions

Add recommonmark to extensions so sphinx project has markdown support.
Configuration details here: https://www.sphinx-doc.org/en/master/usage/markdown.html

* Update source suffix to use latest recommonmark syntax

Updating to the syntax found here: https://www.sphinx-doc.org/en/master/usage/markdown.html

Co-authored-by: Aaron Markham <[email protected]>

* Remove old recommonmark parser syntax

* Pin sphinx version to work with latest version of recommonmark

Co-authored-by: Aaron Markham <[email protected]>

* Pin version latest version recommonmark

Co-authored-by: Aaron Markham <[email protected]>

* Delete theme file generated during build

* de-duplicate install package of psutil (#329)

The Dockerfile of GPU install package psutil duplicated.

* Remove pytorch-sphinx-theme files

* Update to match latest staging_0_1_1

* Update docker directory to match staging_0_1_1 branch

* Delete .DS_Store

* Add sphinxcontrib.katex and torch

Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaqib Ansari <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>
Co-authored-by: Mario Cho <[email protected]>
Co-authored-by: mycpuorg <[email protected]>

* fix for response time out issue during build

* Image classifier example readme fixes (#405)

* Changed the model name in torch.jit.save to be consistent with docs

* Remove extra handler flag and fix the torchservr start

* fix the doc

* fix the doc

Co-authored-by: mycpuorg <[email protected]>

* Porting memory fix changes to TorchServe (#422)

Co-authored-by: dhanainme <[email protected]>

* Benchmark documentation fix (#425)

* Fix Benchmark Documentation

* Documentaion fixes

* .circleci: Add initial circleci configuration (#431)

Signed-off-by: Eli Uriegas <[email protected]>

* Torchscript support huggingface transfrmers (#421)

* Add support for torchscript

* Add support for torchscript

* Add key values for torchscript support

* Add Torchscript suport

* Add Torchscript support

* Add Torchscript support in loading the models

* keep the example to QA

* modify to reflect Torchscript example?

* Add torchscript support guides

* Add fintued BERT for QA

* Add QA artifacts

* Add seqenece classification artifacts

* Token classification artifacts

* Correct typos

* Add max length to tokenizer

* clean up

* enhance the doc

* change setup to seq classification

* Add detail to torchscript support

* set the model for QA

* Add torchscript bert seq classification example

* Add  bert seq classification example

* Modify the packaging model command

* Modify torchscript packaging command

* clean up

* clean up

* correct spacing between list items

Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Integration test huggingface transfromers torchscript support (#433)

* made model server port tests independent

* Add integration tests for Huggingface transfromers with new handler that support torchscript

Co-authored-by: harshbafna <[email protected]>
Co-authored-by: mycpuorg <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Apachebench benchmark - cleanup script / documentation

* Revert "Apachebench benchmark - cleanup script / documentation"

This reverts commit c8f8682.

* Documentation updates for clarity on Eager mode / Torch scripted models (#439)

* Model Zoo updates for Eager Mode / Torchscripted models

* Update examples/README.md

Co-authored-by: Aaqib <[email protected]>

Co-authored-by: Aaqib <[email protected]>

* Apachebench benchmarking - Script / Documentation updates (#438)

* Apachebench benchmarking script - Script / Documentation updates

* Stop TS after benchmark completion.

* Adding system diagram and brief explanation of workings (#413)

* Adding system diagram and brief explanation of workings

* Adding number of worker recommendations

* Adding description of netty configuration params

* Update header for worker threads readme section

Co-authored-by: mycpuorg <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Added TODO statement

* rebased to master

* rebased to master

Co-authored-by: dhaniram-kshirsagar <[email protected]>
Co-authored-by: dhaniram kshirsagar <[email protected]>
Co-authored-by: Aaqib <[email protected]>
Co-authored-by: mycpuorg <[email protected]>
Co-authored-by: dhanainme <[email protected]>
Co-authored-by: mjpsl <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Vamshidhar Dantu <[email protected]>
Co-authored-by: Vamshidhar Dantu <[email protected]>
Co-authored-by: Brad Heintz <[email protected]>
Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>
Co-authored-by: alexwong <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Dhanasekar Karuppasamy <[email protected]>
Co-authored-by: dkasar-aws <[email protected]>
Co-authored-by: Shivam Shriwas <[email protected]>
Co-authored-by: Hamid Shojanazeri <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: shivamshriwas <[email protected]>
Co-authored-by: Takayoshi Nishida <[email protected]>
Co-authored-by: alvations <[email protected]>
Co-authored-by: Tzu-Wei Huang <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Francesco Saverio Zuppichini <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Jessica Lin <[email protected]>
Co-authored-by: Mario Cho <[email protected]>
Co-authored-by: Eli Uriegas <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
htappen pushed a commit to htappen/serve that referenced this pull request Jun 30, 2020
* fixed resnet152 custom handler to work with GPU

* issue template

* Moving issue templates to root folder per pull request template

* adding feature request template

* Minor changes related to sections and names.

* Update feature_template.md

* Update bug_template.md

* doc template

* minor changes

* Minor changes to upload custom model or handler

* issue templates (pytorch#273)

* issue template

* Moving issue templates to root folder per pull request template

* adding feature request template

* Minor changes related to sections and names.

* Update feature_template.md

* Update bug_template.md

* doc template

* minor changes

* Minor changes to upload custom model or handler

Co-authored-by: mycpuorg <[email protected]>

* Fixes pytorch#205 - Docker install not finding GPUs (pytorch#262)

Invokes docker with  --runtime=nvidia for GPU
    Added the option to specify the GPU / specific GPU ids in start.sh
    Fixed the documentation for start.sh script
    Fixed the JDK version in Dockerfile.gpu

Co-authored-by: Aaqib <[email protected]>

* Fix pylint issue (pytorch#163)

* pylint fixes and code cleanup

* UT fixes, pylint updates per latest version and updated sanity script to run pylint

* fixed pylint and wrapt installation issue

* freeze pylint version to 2.4.4 due to bug in version 2.5

* removed duplicate statement to install pylint

Co-authored-by: Aaqib <[email protected]>

* Updated exception handling for worker threads shutdown. (pytorch#172)

* updated code to now log errors only in case when exception is not genreated by unregister api call

* java formatting

* fixed illigalstateexception

* removed the unrequired handling for illegal monitor state exception

Co-authored-by: dhaniram kshirsagar <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* changes for Homebrew  issue 98 (pytorch#126)

* changes for Homebrew  issue 98

* adding changes for path in benchmark.py

* CI Build corrected

* benchmark changes

corrected path

* changed PATH for homebrew

* Update Readme

Made changes for homebrew installation and instruction for GPU based machine instruction for install_dependencies.sh

* Updated Readme.md

removed unnecessory  lines

* corrected indentation

corrected indentation for  new lines added

* Update Readme.md fil

added separate commands for CPU and GPU based

Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: harshbafna <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Harsh Bafna <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* added changes to clean model loaded through URL from model store on unregisted

* added UTs

* Moving the plugins SDK to org.pytorch. Added plugins example. (pytorch#261)

* Moved plugins SDK to org.pytorch

* Updating the repo for publishing plugins SDK

* Enabled bintray maven repo temporarily

* Reset tests

* Added GPG plugin to pom file to sign the plugins SDK

* Update the SDK version

* Removed bintray repo

* Modify the buildspec to not deploy the SDK

Co-authored-by: Vamshidhar Dantu <[email protected]>
Co-authored-by: Aaqib <[email protected]>
Co-authored-by: mycpuorg <[email protected]>

* Added model zoo (pytorch#299)

Co-authored-by: mycpuorg <[email protected]>
Co-authored-by: Brad Heintz <[email protected]>

* Removed snapshot dependency on model (pytorch#152)

* Snapshot is now independent of model

* fixed checkstyle issue

* fixed formatting

* doc review: configuration and custom_service

* addressed review comments

* fixed incorrect merge

* fixed review comments and corrected the error messages

* adding remaining fixes for string constants

Co-authored-by: harshbafna <[email protected]>
Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>
Co-authored-by: Harsh Bafna <[email protected]>

* Regression Tests Suite - AWS Code Build Infra - Initial Commit for pytorch#57 (pytorch#285)

* Regression Tests Suite - AWS Code Build Infra - Initial Commit for pytorch#57

* Script to clone Pytorch Repo & Run Post Man Scripts
* Basic Tests for Inference & Managment APIs
* AWS Code Build buildspec.yml

* Refactor postman tests, add empty postman collection for todos

* Add a readme for adding tests

* Add more management tests

* Fix typo

* Snapshot Regression tests - pytorch#57

* Fix typo

* Comment out snapshot test for now

* Weird typos are showing up, try saving everything

* Need to manually save every request in postman...

* When reregister don't use s3

* More workarounds for rereg

* Fix test messages

* Remove test for empty reqs

* Change get to post

* Fix pytest for subprocess - pytorch#57

* Enable pytest - pytorch#57

* Inference API Testcases / Rename Management Testcases  pytorch#57

* Wipe Model MARs in Modelstore between tests pytorch#54

* Fix Inference Endpoint Execution pytorch#54

* Fix file paths - pytorch#57

* Fix Initial Workers pytorch#57

* Fix Inference Assertions pytorch#57

* Fix DenseNet Inference pytorch#57

* Fix Densenet Register pytorch#54

* Redirect Logs to File pytorch#54

* Fix Test Execution Log pytorch#54

* README, Build Badge pytorch#57

* Format ReadMe pytorch#57

* Update README - Add Instruction to add new tests / Run tests locally

* Update README

* Update branch for build Codebuild Badge

Co-authored-by: alexwong <[email protected]>
Co-authored-by: Aaqib <[email protected]>
Co-authored-by: mycpuorg <[email protected]>
Co-authored-by: Ubuntu <[email protected]>

* Issue 312 install from src (pytorch#322)

* documentation: fixed issues in installation and quick start in README.md

* documentation: Reverting conda related doc changes from PR#286

Fixes pytorch#297

* remove minus y (pytorch#311)

update pip install instructions

* Update instructions for installation from source

This commit aims to address Issue pytorch#312

Tested on a fresh EC2 P3.2 instance:
* Followed instructions under
https://github.com/pytorch/serve#install-torchserve-for-development
* Started the server manually from command line
* Ran a ping query

Received a response from server with "Healthy" status

Signed-off-by: Manoj Rao <[email protected]>

* add separate files for installation

one each for linux and macos

* update README and remove relative path from script

* Install script - Dont uninstall if not previously installed - pytorch#132

* explicitly build the source code with gradle prior to install

* remove upgrade instructions in favor of clean reinstall from source

* update instructions to contain the correct script name

Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaqib Ansari <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>
Co-authored-by: Manoj Rao <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>

* fix for macos script (pytorch#338)

* Move Issue Template to .github dir

* Updated sanity script to use install_from_src_ubuntu script (pytorch#350)

* updated sanity script to use install_from_src_ubuntu script

* added java version check

* fixed issue with if statement

* fixed variable name

* fixed repeated cd command

* updated artifacts

* removed artifacts from buildspec

* Corrected HTTP response code for Model version does not exists exception (pytorch#282)

* issue 196 patch

* Added openapi missing responses. - incorporated review comments

* updated method signature after resolving merge conflicts

Co-authored-by: Harsh Bafna <[email protected]>
Co-authored-by: Aaqib <[email protected]>
Co-authored-by: Shivam Shriwas <[email protected]>
Co-authored-by: harshbafna <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Corrected paths in command and added notes (pytorch#354)

Co-authored-by: dhanainme <[email protected]>

* Refactored UT for better reporting (pytorch#159)

* refactored UTs

* upgraded testng version and updated testcase names

* added testng.xml to handle sequential execution of test classes

* added waveglow example

* Updated readme for waveglow example

* fixed failing UTs after merging master

* removed unintentionally pushed waveglow example

* fixed failing test case

* moved set-default to utils and added couple of more add cases for default version

* java formatting

* corrected set default negative test case

* fixed java formatting

Co-authored-by: Aaqib <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Huggingface Transformers generalization (pytorch#302)

* Scripts to seriliaze a Bert model using jit.trace and a handler for Bert model

* modified to include mapping of predicitons and decoding received text

* move back the original text

* moved the index_to_name to bert folder

* Transformers/ Bert serving with Torchserve readme

* deleting an unused dictionary

* just cleaning some of the commented out parts

* cleaning up

* Cleaning up

* uncommenting the logging and cleaning  the comments

* Download and save the checkpoints, config file and vocab of  the BERT from Transformers for demonstration purposes

* mapping of predicitons to labels

* a custom handler for transformers.

* README.

* sample of input text.

* modifying the descriptions

* modifying the descriptions

* adding comments

* adding commnets

* Fromatting modifications

* modified comments

* added the citations

* adding hugginface as the type of transformer model we are referring to

* renaming the folder to huggingface_transformers

* Removing the bert folder

* simple modification

* modifed the script to use a setup_config.json for loading and saving models

* added the section for transfomer_handler_generalized

* the generalized handler

* set_up config intialization

* edited for transfomer generalization

* modified to use just input_ids as inputs for models

* modified for question answering

* modified to include the number of labels

* corrected typos

* modifed variable names

* edited for clarity

* modifed variable names, simplified the initlialize in the handler class, removed the warnning from QA case

* modifed values for QA

* modifed the variable names

* added comments

* modified to add clarity

* add comment in preprocessing of QA

* added warning for the case mode is not selected properly and combined constructor and initlaize fucntion

* clean up

* clean up

* combined constructor and initliaze func

* combined constructor and initliaze func

* Separated the constructor and initliaze func

* Modified to remove the classification part and update with generalized version

* resolved the coflict and separated the costructor and initliaze func

* removed the handler for classification case as the generalized version handles the case

Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: mycpuorg <[email protected]>

* added waveglow example (pytorch#258)

* added waveglow example

* Updated readme for waveglow example

* enhanced example to remove runtime checkpoint download

* updated handler to return the audio file content instead of dumping it to tmp dir

* minor change to handler

* added pipefail

* added python snippet to run inference

* updated readme with know issue with the example

Co-authored-by: Aaqib <[email protected]>

* made build suite compatible with latest gradle (pytorch#348)

Co-authored-by: dhanainme <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* moved env section after context

* Deleting redundant ISSUE_TEMPLATE at root dir

* Fix typo in scripts/install_from_src_macos

* Corrected  OpenApi description  (pytorch#328)

* Correct query parameter name (Registration Api)in OpenApi description

* removed non existing path parameter

Co-authored-by: Aaqib <[email protected]>

* Regression suite fixes and added test cases (pytorch#344)

* added test model zoo test cases
updated regression script

* Updated Readme

* update -y flag for install

* Updated docker command as per new imagename and tag generated in stagging branch

* Fix sudo usage in regression test script

* - addressed review comments
- reverted changes related to sudo in regression script.

* added https test collection

* - updated failed test cases on staging branch
- added environment and updated newman cmd  with --insecure for https

* updated script

* updated pytest

* update curl check for serve up

* Revert "update curl check for serve up"

This reverts commit 0047eae.

* Revert "update curl check for serve up"

This reverts commit 0047eae.

- update model zoo
- added resnet test case

* added workers to resnet model

* Update readme to run test suite on desired branch

* enhanced inference test cases to driven by datafile

* Updated readme

Co-authored-by: Dhanasekar Karuppasamy <[email protected]>
Co-authored-by: Aaqib <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Fix typo in mnist readme (pytorch#330)

Co-authored-by: Aaqib <[email protected]>

* Fixed typo in ab installation. (pytorch#327)

* documentation: fixed issues in installation and quick start in README.md

* documentation: Reverting conda related doc changes from PR#286

Fixes pytorch#297

* remove minus y (pytorch#311)

update pip install instructions

* Fixed typo in ab installation.

Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaqib Ansari <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>

* Setup Nightly Build for TorchServe (pytorch#371)

* Add Branch Flag for Nightly CodeBuild pytorch#57

* Update ENV Variable for Nightly Build Name

* Nigtly Build - Fix FileName for Build Artifacts

* Update README for accessing test logs

* Whitspace changes

* Build new images for build - README Instruction

* Fix for intermittent gradle build failure (pytorch#352)

* Fix for intermittent null pointer exception

* Fixed comments

Co-authored-by: dhanainme <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* documentation: Remove redundant curl options in example (pytorch#336)

* documentation: fixed issues in installation and quick start in README.md

* documentation: Reverting conda related doc changes from PR#286

Fixes pytorch#297

* remove minus y (pytorch#311)

update pip install instructions

* fix-doc

* fix remaining "-X POST"s

Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaqib Ansari <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>

* Corrected port number validation range (pytorch#304)

* Corrected port number validation range

* added UT for issue 291

* added ut dependency file formatting

* correct param in config_test_env.properties

* removed incorrect flags from test case config files

* added negative test cases

* removed properties file and changes in test case

Co-authored-by: Harsh Bafna <[email protected]>

* Fixes version number reporting for TorchServe (pytorch#360)

* Correct version and added vesion.txt for getting version info

* fixed description test cases and build issue

* Added snapshot test fixes
and resolved relative path issue for version.txt

* python formatting

* added version.txt in MANIFEST.in

* addressed review comments

* added version.txt for model archiver as well
- addressed review comments

* -fixed python UT
-added ts python UT run in sanity script

* removed blank lines from snapshot files

Co-authored-by: Aaqib <[email protected]>
Co-authored-by: Harsh Bafna <[email protected]>

* fixes installation script exits without error if no java present on machine (pytorch#376)

* fixes installation script exits without error if no java present on machine

* added changes for macos script

Co-authored-by: Aaqib <[email protected]>

* Dockerfile enhancements (pytorch#308)

* dockerfile now points to latest torchserve on pypi and added separate files for dev

* Minor changes

* Made build and run commands consistent

* Fixing formatting

* Consistency issue

* Enhanced Dockerfile for binary per pytorch#178 and other misc.

* Fixed gpu related issues in start script

* Corrected the heading font

* Added mar creation steps from container

* added buildkit support for dev docker files

* Fix for buildkit

* Restoring psutil and future as absence leads intermittent hangged build

* Fixed review comments for doc

* Fixed typo

Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* Updated configuration.md for missing config properties (pytorch#337)

* Updated configuration.md for missing parameters

* changes TS_MODEL_SNAPSHOT to MODEL_SNAPSHOT
- removes ambiguity
- follows naming convention

* addressed review comments.

* removed debug property

* updated configuration.md

Co-authored-by: Harsh Bafna <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* Adding missing document links to README (pytorch#379)

* Maven tests should output only errors (pytorch#326)

Co-authored-by: Vamshidhar Dantu <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* fixed code for build failure in python virtual environment (pytorch#341)

* fixed code for build failure in python virtual environment

* added exception missed on resolving conflicts

Co-authored-by: Aaqib <[email protected]>
Co-authored-by: Vamshidhar Dantu <[email protected]>

* updated object detector handler to use only default cuda device for torchvision version less then 0.6 (pytorch#265)

* updated object detector handler to use default cuda device for torchvision version 0.5.0

* updated version check logic

* Updated documentation for torchvision version dependency

* enhanced sanity script to validate all default handlers

* fixed sanity script

* added torchtext install step in sanity script

* fixed issue related to inference after restart with snapshot.

* moved torchtext to codebuild docker files

* added more log statements

* enhanced run inference function

* fixed register model function

* added unregister model function

* incorporated code review comments

* added back inference call for snapshot test

* fixed if block

Co-authored-by: Aaqib <[email protected]>

* Adding a barebones ec2 template to run torchserve on ubuntu (pytorch#301)

* Adding a barebones ec2 template to run torchserve on ubuntu

* Adding VPC with Private subnet

* Address review comments #1

* Address review comments pytorch#2

* Rebase for pytorch#365 (pytorch#389)

* Updated README.md for pytorch#364

See pytorch#364 for a deep description of the problem

* updated README.md

* updated README.md

Co-authored-by: Francesco Saverio Zuppichini <[email protected]>

* Document the docker settings for running in production mode (pytorch#388)

* Docker Production Options pytorch#234

* Docker Production Options - Typo Fixes / Fit & Finish  pytorch#234

* Typos - Docker Docs pytorch#234

* grammar check  - docker/README.md

Co-authored-by: Aaqib <[email protected]>

* grammar check  - docker/README.md

Co-authored-by: Aaqib <[email protected]>

* grammar check  - docker/README.md

Co-authored-by: Aaqib <[email protected]>

Co-authored-by: Aaqib <[email protected]>

* Huggingface transformers integration test (pytorch#382)

* Add Huggingface transformers example integration tests

* Add transformers intallation

* Fix a missing bracket

* move transformers installation to chain installations

Co-authored-by: Geeta Chauhan <[email protected]>

* Updated docker README.md (pytorch#401)

* Update README.md

* Fixed cd command

* Made source cloning steps explicit!!

* Fixed online editing mistake

Co-authored-by: Geeta Chauhan <[email protected]>

* Fixed response status message for inferencing on model with zero workers (pytorch#305)

* Changes for issue_29

* Changed error Messages

* changed message for scalling request

* changed defaultVersion string

* Corrected error message

* changed the code for best practice

* resolved review comment for version

* updated management api doc as per new responses in worker scale api

Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Harsh Bafna <[email protected]>
Co-authored-by: dhanainme <[email protected]>
Co-authored-by: Shivam Shriwas <[email protected]>
Co-authored-by: Aaqib <[email protected]>

* made model server port tests independent (pytorch#396)

* Added backend profiling and fixed benchmark's install dependency scripts (pytorch#402)

* added backend profiling
* updated install dependency scripts to make them jmeter version independent
* updated readme
* Fix benchmarks/README typo and JMeter versioning issue
* updated documentation

Co-authored-by: Aaqib Ansari <[email protected]>

* Add sphinx directory and conf.py (pytorch#274)

* Add sphinx folder that contains files for docs build

* Delete .DS_Store

* Delete .DS_Store

* Add conf.py in docs/

* documentation: fixed issues in installation and quick start in README.md

* documentation: Reverting conda related doc changes from PR#286

Fixes pytorch#297

* remove minus y (pytorch#311)

update pip install instructions

* Update to the latest version of sphinx

* Add recommonmark to extensions

Add recommonmark to extensions so sphinx project has markdown support.
Configuration details here: https://www.sphinx-doc.org/en/master/usage/markdown.html

* Update source suffix to use latest recommonmark syntax

Updating to the syntax found here: https://www.sphinx-doc.org/en/master/usage/markdown.html

Co-authored-by: Aaron Markham <[email protected]>

* Remove old recommonmark parser syntax

* Pin sphinx version to work with latest version of recommonmark

Co-authored-by: Aaron Markham <[email protected]>

* Pin version latest version recommonmark

Co-authored-by: Aaron Markham <[email protected]>

* Delete theme file generated during build

* de-duplicate install package of psutil (pytorch#329)

The Dockerfile of GPU install package psutil duplicated.

* Remove pytorch-sphinx-theme files

* Update to match latest staging_0_1_1

* Update docker directory to match staging_0_1_1 branch

* Delete .DS_Store

* Add sphinxcontrib.katex and torch

Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaqib Ansari <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>
Co-authored-by: Mario Cho <[email protected]>
Co-authored-by: mycpuorg <[email protected]>

* Image classifier example readme fixes (pytorch#405)

* Changed the model name in torch.jit.save to be consistent with docs

* Remove extra handler flag and fix the torchservr start

* fix the doc

* fix the doc

Co-authored-by: mycpuorg <[email protected]>

* Porting memory fix changes to TorchServe (pytorch#422)

Co-authored-by: dhanainme <[email protected]>

* Benchmark documentation fix (pytorch#425)

* Fix Benchmark Documentation

* Documentaion fixes

* .circleci: Add initial circleci configuration (pytorch#431)

Signed-off-by: Eli Uriegas <[email protected]>

* Torchscript support huggingface transfrmers (pytorch#421)

* Add support for torchscript

* Add support for torchscript

* Add key values for torchscript support

* Add Torchscript suport

* Add Torchscript support

* Add Torchscript support in loading the models

* keep the example to QA

* modify to reflect Torchscript example?

* Add torchscript support guides

* Add fintued BERT for QA

* Add QA artifacts

* Add seqenece classification artifacts

* Token classification artifacts

* Correct typos

* Add max length to tokenizer

* clean up

* enhance the doc

* change setup to seq classification

* Add detail to torchscript support

* set the model for QA

* Add torchscript bert seq classification example

* Add  bert seq classification example

* Modify the packaging model command

* Modify torchscript packaging command

* clean up

* clean up

* correct spacing between list items

Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Integration test huggingface transfromers torchscript support (pytorch#433)

* made model server port tests independent

* Add integration tests for Huggingface transfromers with new handler that support torchscript

Co-authored-by: harshbafna <[email protected]>
Co-authored-by: mycpuorg <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: dhanainme <[email protected]>

* Apachebench benchmark - cleanup script / documentation

* Revert "Apachebench benchmark - cleanup script / documentation"

This reverts commit c8f8682.

* Documentation updates for clarity on Eager mode / Torch scripted models (pytorch#439)

* Model Zoo updates for Eager Mode / Torchscripted models

* Update examples/README.md

Co-authored-by: Aaqib <[email protected]>

Co-authored-by: Aaqib <[email protected]>

* Apachebench benchmarking - Script / Documentation updates (pytorch#438)

* Apachebench benchmarking script - Script / Documentation updates

* Stop TS after benchmark completion.

Co-authored-by: harshbafna <[email protected]>
Co-authored-by: dhaniram-kshirsagar <[email protected]>
Co-authored-by: dhaniram kshirsagar <[email protected]>
Co-authored-by: Aaqib <[email protected]>
Co-authored-by: mycpuorg <[email protected]>
Co-authored-by: Harsh Bafna <[email protected]>
Co-authored-by: mjpsl <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Vamshidhar Dantu <[email protected]>
Co-authored-by: Vamshidhar Dantu <[email protected]>
Co-authored-by: Brad Heintz <[email protected]>
Co-authored-by: eslesar-aws <[email protected]>
Co-authored-by: Aaron Markham <[email protected]>
Co-authored-by: alexwong <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Geeta Chauhan <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: dkasar-aws <[email protected]>
Co-authored-by: Shivam Shriwas <[email protected]>
Co-authored-by: Hamid Shojanazeri <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: shivamshriwas <[email protected]>
Co-authored-by: Takayoshi Nishida <[email protected]>
Co-authored-by: alvations <[email protected]>
Co-authored-by: Tzu-Wei Huang <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Francesco Saverio Zuppichini <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Jessica Lin <[email protected]>
Co-authored-by: Mario Cho <[email protected]>
Co-authored-by: Eli Uriegas <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
@datavistics
Copy link

Its called max_request_size in the config.properties file.
Here is what Im using:
max_request_size=999999999

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.

enable_envvars_config appears to do nothing
7 participants