-
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(traffic_light_visualizer): remove cerr temporarily to avoid flooding logs #8294
fix(traffic_light_visualizer): remove cerr temporarily to avoid flooding logs #8294
Conversation
…ing logs Signed-off-by: kminoda <koji.minoda@tier4.jp>
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
Signed-off-by: kminoda <koji.minoda@tier4.jp>
@yukkysaito Hi, would you briefly check this PR? |
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.
LGTM.
- I confirmed that this PR's content is good to merge.
- I did not check this PR by running autoware.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8294 +/- ##
==========================================
- Coverage 29.31% 0.00% -29.32%
==========================================
Files 1609 13 -1596
Lines 119413 442 -118971
Branches 51480 57 -51423
==========================================
- Hits 35000 0 -35000
+ Misses 75112 442 -74670
+ Partials 9301 0 -9301
☔ View full report in Codecov by Sentry. |
@kminoda I think this PR can be merged. |
…ing logs (autowarefoundation#8294) * fix(traffic_light_visualizer): remove cerr temporarily to avoid flooding logs Signed-off-by: kminoda <koji.minoda@tier4.jp> * fix precommit Signed-off-by: kminoda <koji.minoda@tier4.jp> * fix Signed-off-by: kminoda <koji.minoda@tier4.jp> --------- Signed-off-by: kminoda <koji.minoda@tier4.jp>
Description
Addresses #7924 as a very short term way, as this issue is critical in our environment
Related links
#7924
How was this PR tested?
Not tested since I could not even reproduce the flooding error messages.
Notes for reviewers
None.
Interface changes
None.
Effects on system behavior
None.