rospy.log*_throttle (and _once) do not accept (kw)args #1222
Comments
|
@wkentaro Maybe you can answer the question and/or provide a follow up PR to align the signatures. |
mgrrx
added a commit
to magazino/ros_comm
that referenced
this issue
Jan 1, 2018
Implements ros#1222
mgrrx
added a commit
to magazino/ros_comm
that referenced
this issue
Sep 21, 2018
Implements ros#1222
mgrrx
added a commit
to magazino/ros_comm
that referenced
this issue
Sep 21, 2018
Implements ros#1222
dirk-thomas
pushed a commit
that referenced
this issue
Feb 10, 2020
* Added args and kwargs to rospy.log* Implements #1222 * Exposed logger_name * Fixup
|
Addressed by #1290. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
It would appear the
rospy.logX_throttle(..)androspy.logX_once(..)methods introduced in #812 and #1041 do not accept args or keyword args like the regularlogX(..)methods do.This means that instead of
rospy.loginfo("fmt", arg0, arg1, argN)they have to be passed a bare string (ie:rospy.loginfo_throttle(1.0, "some string")).This can be worked around by using
str.format(..)itself, but symmetry with the baselogX(..)methods is lost this way.Was this intentional, or an oversight?
The text was updated successfully, but these errors were encountered: