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.

Sign up for GitHub

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

Jump to bottom

[serving] Fix flaky test #1013

Merged
merged 1 commit into from
Jun 11, 2021
Merged

[serving] Fix flaky test #1013

merged 1 commit into from
Jun 11, 2021

Conversation

frankfliu
Copy link
Contributor

Change-Id: I13b89e04516c59a3d28ecafd49f4f808630b22fb

Description

Brief description of what this PR is about

  • If this change is a backward incompatible change, why must this change be made?
  • Interesting edge cases to note here

@frankfliu
[serving] Fix flaky test
46f6bff 
Change-Id: I13b89e04516c59a3d28ecafd49f4f808630b22fb
@codecov-commenter
Copy link

codecov-commenter commented Jun 11, 2021 •
edited
Loading

Codecov Report

Merging #1013 (46f6bff) into master (77809f4) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1013   +/-   ##
=========================================
  Coverage     69.97%   69.97%           
  Complexity     5106     5106           
=========================================
  Files           504      504           
  Lines         22601    22601           
  Branches       2367     2367           
=========================================
  Hits          15815    15815           
  Misses         5519     5519           
  Partials       1267     1267

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 77809f4...46f6bff. Read the comment docs.

@frankfliu frankfliu merged commit 3379fd2 into deepjavalibrary:master Jun 11, 2021
@frankfliu frankfliu deleted the flaky branch June 11, 2021 21:53
AzizZayed added a commit to AzizZayed/djl that referenced this pull request Jun 15, 2021
@AzizZayed
Squashed commit of the following:
697655b 
commit 0092f8e
Author: Aziz Zayed <azayed01@gmail.com>
Date:   Tue Jun 15 08:22:51 2021 -0700

    Fixed truncated-normal bug

commit a6ded8c
Author: Aziz Zayed <azayed01@gmail.com>
Date:   Mon Jun 14 13:33:30 2021 -0700

    [pytorch] Add BigGAN demo

commit f145614
Merge: a8a1a9b ec8405b
Author: Abd-El-Aziz Zayed <48853777+AzizZayed@users.noreply.github.com>
Date:   Fri Jun 11 20:45:34 2021 -0700

    Merge branch 'deepjavalibrary:master' into master

commit ec8405b
Author: Abd-El-Aziz Zayed <48853777+AzizZayed@users.noreply.github.com>
Date:   Fri Jun 11 14:53:59 2021 -0700

    [pytorch] Add oneHot operator (deepjavalibrary#1014)

    [tensoflow] Add truncated normal operation

commit 50600fd
Author: Frank Liu <frankfliu2000@gmail.com>
Date:   Fri Jun 11 14:53:43 2021 -0700

    upgrade dependencies version (deepjavalibrary#1012)

    Change-Id: I709938f69f21096bc5cd29a24191f0f282dcbc97

commit 3379fd2
Author: Frank Liu <frankfliu2000@gmail.com>
Date:   Fri Jun 11 14:53:29 2021 -0700

    [serving] Fix flaky test (deepjavalibrary#1013)

    Change-Id: I13b89e04516c59a3d28ecafd49f4f808630b22fb

commit 23157fd
Author: Frank Liu <frankfliu2000@gmail.com>
Date:   Thu Jun 10 16:31:03 2021 -0700

    Enable spotbugs for java 11+ (deepjavalibrary#1010)

    Change-Id: I74effbf45492a5cf50e09ba8af0223d2b1bcb5a5

commit 4f38708
Author: Frank Liu <frankfliu2000@gmail.com>
Date:   Thu Jun 10 16:30:50 2021 -0700

    Fix model zoo test typo (deepjavalibrary#1009)

    Change-Id: I7c0109c6e5fc0ece16288082fd830718f20ad489

commit a8a1a9b
Merge: 77809f4 30b03f4
Author: Aziz Zayed <azayed01@gmail.com>
Date:   Thu Jun 10 15:16:05 2021 -0700

    Merge Truncated-Normal branch

commit 77809f4
Author: Frank Liu <frankfliu2000@gmail.com>
Date:   Thu Jun 10 14:07:43 2021 -0700

    Make model zoo test weekly (deepjavalibrary#1004)

    Change-Id: I1c73df17cb077b9ce8905fcc2fc8bbb37b9688d8

commit 0aec8ca
Author: Abd-El-Aziz Zayed <48853777+AzizZayed@users.noreply.github.com>
Date:   Thu Jun 10 12:46:16 2021 -0700

    [tensoflow] Add truncated normal operation (deepjavalibrary#1005)

commit 30b03f4
Author: Aziz Zayed <azayed01@gmail.com>
Date:   Wed Jun 9 01:40:33 2021 -0700

    [tensoflow] Add truncated normal operation

commit d8e7e1d
Author: Frank Liu <frankfliu2000@gmail.com>
Date:   Wed Jun 9 07:55:15 2021 -0700

    Fixes deepjavalibrary#999, hanlde UTF16 surrogate charactors properly. (deepjavalibrary#1003)

    Change-Id: I19e77cf5a8282bea901434041806eb102549ec0f

commit b0fe73a
Author: Frank Liu <frankfliu2000@gmail.com>
Date:   Tue Jun 8 17:56:19 2021 -0700

    [pytorch] Update load model jupyter notebook (deepjavalibrary#1002)

    Change-Id: I1889aa93d2002e6ce02c740d2d1d3517bf586760

commit 8286930
Author: Frank Liu <frankfliu2000@gmail.com>
Date:   Tue Jun 8 15:29:27 2021 -0700

    [tensorflow] fix optOption usage document (deepjavalibrary#1001)

    Change-Id: Ie044839cf082d63010a5c26d3f2f8833447919c6

commit a26f5b2
Author: Abd-El-Aziz Zayed <48853777+AzizZayed@users.noreply.github.com>
Date:   Tue Jun 8 15:29:10 2021 -0700

    Updated PyTorch Docs  (deepjavalibrary#1000)

    * Added auto softmax metadata for action_recognition

    * Update PyTorch Docs

commit e6890f9
Author: Lanking <qingla@amazon.com>
Date:   Mon Jun 7 18:25:19 2021 -0700

    upgrade xgboost (deepjavalibrary#993)

commit a0dcf3a
Author: Lanking <qingla@amazon.com>
Date:   Mon Jun 7 18:25:12 2021 -0700

    bump up onnx runtime version (deepjavalibrary#992)
Lokiiiiii pushed a commit to Lokiiiiii/djl that referenced this pull request Oct 10, 2023
@frankfliu
[serving] Return proper HTTP status code for each batch (deepjavalibr…
0f9a4d3 
…ary#1013)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stu1130 stu1130 stu1130 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@frankfliu @codecov-commenter @stu1130