wg-perception / object_recognition_core

The core of object recognition, where the development now happens
48 stars 47 forks source link

Corrected error: #16

Closed destogl closed 11 years ago

destogl commented 11 years ago

When calling:

rosrun object_recognition_core detection -c `rospack find object_recognition_tod`/conf/detection.ros.ork --visualize

got this error:

Traceback (most recent call last):
  File "/home/destogl/Workspace/ros/catkin_ws/src/object_recognition_core/apps/detection", line 21, in <module>
    plasm = create_plasm(ork_params)
  File "/home/destogl/Workspace/ros/catkin_ws/src/object_recognition_core/python/object_recognition_core/pipelines/plasm.py", line 66, in create_plasm
    cells[cell_name] = cells[cell_name](cell_name=cell_name, n_inputs=n_inputs, **ork_params[cell_name])
  File "/home/destogl/Workspace/ros/catkin_ws/src/object_recognition_core/python/object_recognition_core/io/voter.py", line 37, in __init__
    ecto.BlackBox.__init__(self, *args, **kwargs)
  File "/opt/ros/groovy/lib/python2.7/dist-packages/ecto/blackbox.py", line 248, in __init__
    self.__class__.__name__) + 'needs to be decorated with @classmethod or @staticmethod.')
ecto.blackbox.BlackBoxError: The "declare_cells" function of the BlackBox Aggregator needs to be decorated with @classmethod or @staticmethod.

This happens because inherited methods have attribute @classmethod .

vrabaud commented 11 years ago

Thx, I just fixed that one too and reenabled some ROS tests : I had disabled those because the ROS celles required ROS to be launched at some point (and the rostest macros were not there yet). It's all good now, sorry for the disruption and thx for the help !

destogl commented 11 years ago

When will you push your fix?

vrabaud commented 11 years ago

that was done in 54d83d0054ebedbf77a14d1ecac2526395845478 . Do you mean a package release ?

destogl commented 11 years ago

Sorry I didn't see that!