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
[rostwitter] Suppress tweet log #378
[rostwitter] Suppress tweet log #378
Conversation
rostwitter/scripts/tweet.py
Outdated
@@ -28,7 +28,7 @@ def __init__(self): | |||
|
|||
def tweet_cb(self, msg): | |||
message = msg.data | |||
rospy.loginfo(rospy.get_name() + " sending %s", message) | |||
rospy.logdebug(rospy.get_name() + " sending %s", message) |
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.
if loginfo
is not noisy without bsae64 message, how abotut use loginfo + something like ' '.join([a if len(a) < 32 else a[0:32]+'...' for a in txt.split(' ')])
?
If we already feel darty without base64, using logdebug
make sense
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.
Thank you for advice. I think loginfo is not noisy without base64 message.
So, I fixed it in 6451728
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.
Thakyou
Duplicates of knorth55#33
This PR suppresses tweet log.
If Base64 encoding texts are displayed in logs, logs are too dirty to see.