Skip to content

Commit 65e0e61

Browse files
ijnekmergify[bot]
authored andcommitted
improve type readability in errors (#469)
Signed-off-by: ijnek <kenjibrameld@gmail.com> (cherry picked from commit 9c5f6a7)
1 parent ecff92a commit 65e0e61

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

launch_ros/launch_ros/parameter_descriptions.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ def value_type(self) -> AllowedTypesType:
8383
def __str__(self) -> Text:
8484
return (
8585
'launch_ros.description.ParameterValue'
86-
f'(value={self.value}, value_type={self.value_type})'
86+
f'(value={self.value}, value_type={self.value_type.__name__})'
8787
)
8888

8989
def evaluate(self, context: LaunchContext) -> 'EvaluatedParameterValue':
@@ -148,7 +148,7 @@ def value_type(self) -> AllowedTypesType:
148148
def __str__(self) -> Text:
149149
return (
150150
'launch_ros.description.Parameter'
151-
f'(name={self.name}, value={self.value}, value_type={self.value_type})'
151+
f'(name={self.name}, value={self.value}, value_type={self.value_type.__name__})'
152152
)
153153

154154
def evaluate(self, context: LaunchContext) -> Tuple[Text, 'EvaluatedParameterValue']:

launch_ros/launch_ros/utilities/evaluate_parameters.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ def check_sequence_type_is_allowed(sequence):
104104
'If the parameter is meant to be a string, try wrapping it in '
105105
'launch_ros.parameter_descriptions.ParameterValue'
106106
'(value, value_type=str)'.format(
107-
type(yaml_evaluated_value),
107+
type(yaml_evaluated_value).__name__,
108108
evaluated_name
109109
)
110110
)

0 commit comments

Comments
 (0)