-
Notifications
You must be signed in to change notification settings - Fork 630
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
fix(tensorrt_yolo): change the CMakeLists.txt folders for compiling error on Jetson AGX-Orin #1436
Conversation
@@ -5,7 +5,8 @@ find_package(autoware_cmake REQUIRED) | |||
autoware_package() | |||
|
|||
option(CUDA_VERBOSE "Verbose output of CUDA modules" OFF) | |||
|
|||
set(DEPRECATION_FLAG "-Wno-error=deprecated-declarations") | |||
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${DEPRECATION_FLAG}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${DEPRECATION_FLAG}" | |
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${DEPRECATION_FLAG}") |
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unnecessary change
@@ -5,7 +5,8 @@ find_package(autoware_cmake REQUIRED) | |||
autoware_package() | |||
|
|||
option(CUDA_VERBOSE "Verbose output of CUDA modules" OFF) | |||
|
|||
set(DEPRECATION_FLAG "-Wno-error=deprecated-declarations") | |||
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${DEPRECATION_FLAG}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${DEPRECATION_FLAG}" | |
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${DEPRECATION_FLAG}") |
@@ -7,7 +7,8 @@ autoware_package() | |||
option(CUDA_VERBOSE "Verbose output of CUDA modules" OFF) | |||
|
|||
find_package(OpenCV REQUIRED) | |||
|
|||
set(DEPRECATION_FLAG "-Wno-error=deprecated-declarations") | |||
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${DEPRECATION_FLAG}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${DEPRECATION_FLAG}" | |
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${DEPRECATION_FLAG}") |
@@ -5,6 +5,9 @@ find_package(autoware_cmake REQUIRED) | |||
autoware_package() | |||
|
|||
option(CUDA_VERBOSE "Verbose output of CUDA modules" OFF) | |||
|
|||
set(DEPRECATION_FLAG "-Wno-error=deprecated-declarations") | |||
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${DEPRECATION_FLAG}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${DEPRECATION_FLAG}" | |
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${DEPRECATION_FLAG}") |
@@ -7,7 +7,8 @@ autoware_package() | |||
option(CUDA_VERBOSE "Verbose output of CUDA modules" OFF) | |||
|
|||
find_package(OpenCV REQUIRED) | |||
|
|||
set(DEPRECATION_FLAG "-Wno-error=deprecated-declarations") | |||
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${DEPRECATION_FLAG}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${DEPRECATION_FLAG}" | |
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${DEPRECATION_FLAG}") |
Codecov Report
@@ Coverage Diff @@
## main #1436 +/- ##
========================================
- Coverage 9.59% 9.55% -0.04%
========================================
Files 1042 1047 +5
Lines 71837 72106 +269
Branches 15590 15590
========================================
Hits 6890 6890
- Misses 58978 59247 +269
Partials 5969 5969
*This pull request uses carry forward flags. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
@wep21 can you approve it if it is ok now? |
…d bring back to timer trigger autowarefoundation#8277 (autowarefoundation#1436) * fix: revert latency reduction logic and bring back to timer trigger Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp> * style(pre-commit): autofix * chore: remove unused variables Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp> --------- Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Description
Jetson AGX-Orin uses Jetpack 5.1.0 which includes:
These library versions were giving deprecation errors while compiling some packages:
To fix this issue, compiler flag added to CMakeLists.txt. Should we add this situation to documantation? I opened an issue in autoware-documentation for this case.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.
In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.
Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.
After all checkboxes are checked, anyone who has write access can merge the PR.