Skip to content

Handle Dropout_ and ABI fixes for C++ and Python #126

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
Merged
merged 9 commits into from
Jul 9, 2020
Merged

Handle Dropout_ and ABI fixes for C++ and Python #126

merged 9 commits into from
Jul 9, 2020

Conversation

narendasan
Copy link
Collaborator

Description

Adds the option to build libtrtorch.tar.gz with pre-cxx11-abi and for python to build with the CXX11 ABI.

Type of change

Please delete options that are not relevant and/or add your own.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas and hacks
  • I have made corresponding changes to the documentation and have regenerated the documentation (make html in docsrc)
  • I have added tests to verify my fix or my feature
  • New and existing unit tests pass locally with my changes

narendasan and others added 9 commits June 11, 2020 11:04
@narendasan
fix(//cpp/ptq): Enable FP16 kernels for INT8 applications
e1c5416 
Signed-off-by: Naren Dasan <[email protected]>
Signed-off-by: Naren Dasan <[email protected]>
@narendasan
Merge branch 'master' of https://github.com/NVIDIA/TRTorch
a4c8511 
Signed-off-by: Naren Dasan <[email protected]>
@narendasan
Merge branch 'master' of https://github.com/NVIDIA/TRTorch
19c91f2
@narendasan
feat(aten::dropout_): Remove inplace dropout
7aa57c3 
Signed-off-by: Naren Dasan <[email protected]>
Signed-off-by: Naren Dasan <[email protected]>
@narendasan
Merge branch 'master' of https://github.com/NVIDIA/TRTorch
4c23ed2
@narendasan
Merge branch 'master' of https://github.com/NVIDIA/TRTorch
a22de47
@narendasan
Merge branch 'master' of https://github.com/NVIDIA/TRTorch
58006ba
@narendasan
fix(//cpp/trtorchc): Support building trtorchc with the pre_cxx11_abi
172d4d5 
for libtrtorch building with --config=pre_cxx11_abi

Signed-off-by: Naren Dasan <[email protected]>
Signed-off-by: Naren Dasan <[email protected]>
@narendasan
feat(//py): add the option to build python package with CXX11 abi
fdbd7d2 
Signed-off-by: Naren Dasan <[email protected]>
Signed-off-by: Naren Dasan <[email protected]>
@github-actions github-actions bot added component: conversion Issues re: Conversion stage component: lowering Issues re: The lowering / preprocessing passes labels Jul 9, 2020
@narendasan narendasan merged commit aab80bf into master Jul 9, 2020
@narendasan narendasan deleted the python_abi_option branch July 9, 2020 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
component: conversion Issues re: Conversion stage component: lowering Issues re: The lowering / preprocessing passes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@narendasan