Skip to content

BERT demo integration for TensorRT-7.0 - #420

Merged
nvpstr merged 9 commits into
NVIDIA:masterfrom
rajeevsrao:dev/master-trt-bert-7.0
May 11, 2020
Merged

BERT demo integration for TensorRT-7.0#420
nvpstr merged 9 commits into
NVIDIA:masterfrom
rajeevsrao:dev/master-trt-bert-7.0

Conversation

@rajeevsrao

Copy link
Copy Markdown
Contributor

Signed-off-by: Rajeev Rao rajeevrao@nvidia.com

@lgtm-com

lgtm-com Bot commented Mar 12, 2020

Copy link
Copy Markdown

This pull request introduces 12 alerts and fixes 3 when merging a695ffb into b03375b - view on LGTM.com

new alerts:

  • 7 for Unused local variable
  • 4 for Unreachable code
  • 1 for Unused import

fixed alerts:

  • 1 for Unreachable code
  • 1 for __del__ is called explicitly
  • 1 for Variable defined multiple times

@nvpstr

nvpstr commented Mar 20, 2020

Copy link
Copy Markdown
Contributor

@rajeevsrao can you fix the alerts from lgtm?

@lgtm-com

lgtm-com Bot commented Mar 29, 2020

Copy link
Copy Markdown

This pull request introduces 4 alerts and fixes 3 when merging 59a6dfc into dd83ae1 - view on LGTM.com

new alerts:

  • 3 for Unused local variable
  • 1 for Unused import

fixed alerts:

  • 1 for Unreachable code
  • 1 for __del__ is called explicitly
  • 1 for Variable defined multiple times

@rajeevsrao
rajeevsrao force-pushed the dev/master-trt-bert-7.0 branch from 59a6dfc to d79df6a Compare March 29, 2020 11:39
@lgtm-com

lgtm-com Bot commented Mar 29, 2020

Copy link
Copy Markdown

This pull request introduces 1 alert and fixes 3 when merging d79df6a into dd83ae1 - view on LGTM.com

new alerts:

  • 1 for Unused import

fixed alerts:

  • 1 for Unreachable code
  • 1 for __del__ is called explicitly
  • 1 for Variable defined multiple times

@lgtm-com

lgtm-com Bot commented Mar 30, 2020

Copy link
Copy Markdown

This pull request introduces 3 alerts and fixes 3 when merging a2a061f into 9492dc7 - view on LGTM.com

new alerts:

  • 1 for Unused local variable
  • 1 for Unused import
  • 1 for Unreachable code

fixed alerts:

  • 1 for Unreachable code
  • 1 for __del__ is called explicitly
  • 1 for Variable defined multiple times

@lgtm-com

lgtm-com Bot commented Mar 30, 2020

Copy link
Copy Markdown

This pull request introduces 1 alert and fixes 3 when merging da95dd2 into 9492dc7 - view on LGTM.com

new alerts:

  • 1 for Unused import

fixed alerts:

  • 1 for Unreachable code
  • 1 for __del__ is called explicitly
  • 1 for Variable defined multiple times

@lgtm-com

lgtm-com Bot commented Apr 4, 2020

Copy link
Copy Markdown

This pull request introduces 1 alert and fixes 3 when merging 7bc7ce1 into c742ed4 - view on LGTM.com

new alerts:

  • 1 for Unused import

fixed alerts:

  • 1 for Unreachable code
  • 1 for __del__ is called explicitly
  • 1 for Variable defined multiple times

@rajeevsrao
rajeevsrao force-pushed the dev/master-trt-bert-7.0 branch from 70efdb5 to 20319fe Compare April 20, 2020 19:13
Signed-off-by: Rajeev Rao <rajeevrao@nvidia.com>
Signed-off-by: Rajeev Rao <rajeevrao@nvidia.com>
Signed-off-by: Rajeev Rao <rajeevrao@nvidia.com>
- Revert network to use fcPlugin and geluPlugin

Signed-off-by: Rajeev Rao <rajeevrao@nvidia.com>
Signed-off-by: Rajeev Rao <rajeevrao@nvidia.com>
@rajeevsrao
rajeevsrao force-pushed the dev/master-trt-bert-7.0 branch from 20319fe to c608b65 Compare April 21, 2020 00:35
@nvpstr
nvpstr merged commit 8fecbe7 into NVIDIA:master May 11, 2020
PeganovAnton pushed a commit to PeganovAnton/DeepLearningExamples that referenced this pull request Sep 8, 2020
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.

2 participants