Ask Your Question
2

dynamic_reconfigure (python) changes in fuerte

asked 2012-06-14 06:46:42 -0500

brice rebsamen gravatar image

I recently migrated to fuerte and noticed that the sick_ldmrs driver is broken due to some changes in dynamic_reconfigure python's API in fuerte.

"apply_changes" in config

had to be changed to

"apply_changes" in config.__dict__

,

self.ros_params = config.copy()

had to be changed to

self.ros_params = copy.deepcopy(config.__dict__)

,

for param_name, value in config.iteritems()

had to be changed to

for param_name, value in config.items()

Accessing the __dict__ from outside of the class seems an ugly trick to me. Because it's a super-private variable... Is there any other way this could be achieved?

edit retag flag offensive close merge delete

2 Answers

Sort by ยป oldest newest most voted
1

answered 2012-06-24 23:59:48 -0500

joq gravatar image

That looks like a bug to me.

To get it fixed, please open a defect ticket, which can link back to this question.

edit flag offensive delete link more

Comments

1
brice rebsamen gravatar image brice rebsamen  ( 2012-06-25 03:02:16 -0500 )edit
2

answered 2012-06-25 07:33:44 -0500

klapow gravatar image

Basically this was a defect in some of the code I wrote last year to support parameter groups and has been fixed in the latest version of dynamic_reconfigure which you can get here and will be in the next release of dynamic_reconfigure.

edit flag offensive delete link more

Comments

Got the fix, sickldmrs is working out of the box now.

brice rebsamen gravatar image brice rebsamen  ( 2012-08-07 00:38:54 -0500 )edit

Your Answer

Please start posting anonymously - your entry will be published after you log in or create a new account.

Add Answer

Question Tools

1 follower

Stats

Asked: 2012-06-14 06:46:42 -0500

Seen: 240 times

Last updated: Jun 25 '12