-
Notifications
You must be signed in to change notification settings - Fork 672
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
feat: backwardcpp test #6796
feat: backwardcpp test #6796
Conversation
@xmfcx Thank you, Fatih. This looks great. I have two questions: I set backwardcpp in the behavior_path_planner, but it didn't provide any stack trace. I suspect the ros-container, but are you familiar with how to make it work in the container? Additionally, when I set the following code in the std::vector<int> v;
std::cerr << "v = " << v.back() << std::endl; I've got this stack trace from backwardcpp. I expected the stack trace expands Note: I built these packages with RelWithDebInfo option. Click to expand
|
Note: I can check the error logger with kill command: This is the output for the map_based_prediction. Click to expand
|
@TakaHoribe did you compile with debug flags? ( From your post:
It says you got the error on line 902. Could you share the commit so we can analyze? |
I will look into it, I didn't try it with a container before. |
@xmfcx Thank you for your fast reply...! Oh, I didn't see this part This is the branch I'm working on: https://github.com/tier4/autoware.universe/blob/backwardcpp-prediction-test/perception/map_based_prediction/src/map_based_prediction_node.cpp/#L902 |
Also, did you compile with debug flags? (--cmake-args -DCMAKE_BUILD_TYPE=RelWithDebInfo) This is what I used: |
Yes, I tested with |
@xmfcx In my test,
So, if there is a way to use backward-ros, the backward-ros is preferred. Release RelWithDebInfo backward-ros: function name and line information (don't know why but I couldn't get the source code info)Note: setting both |
@xmfcx Hi, is this PR still intended to be open? |
Description
This is for demonstration of backward-cpp with backward_ros package with autoware.
Also recommended by Nav 2 Guide and Davide Faconti 😄
@TakaHoribe this is continuation from:
Please test in both branches:
glog
Crash logs with glog:
Click to expand
backward-cpp
Crash logs with backward-cpp:
Click to expand
Effects on system behavior
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.