Commit 217443c0 authored by Sergi Hernandez's avatar Sergi Hernandez
Browse files

Solved a bug in the read_params functions: The bools parameters was used for all data types.

parent cd176710
......@@ -170,7 +170,7 @@ class CModule
* \param name prefix given to the module servie variables in order to locate and
* handle them.
*/
void dynamic_reconfigure(ModuleCfg &config, const std::string &name);
virtual void dynamic_reconfigure(ModuleCfg &config, const std::string &name);
/**
* \brief function to start the operation of the class
*
......
......@@ -336,11 +336,11 @@ void CModuleDynReconf<dyn_reconf_config>::read_params(void)
for(i=0;i<set_params_srv.response.config.bools.size();i++)
this->bool_params[set_params_srv.response.config.bools[i].name]=set_params_srv.response.config.bools[i].value;
for(i=0;i<set_params_srv.response.config.ints.size();i++)
this->bool_params[set_params_srv.response.config.ints[i].name]=set_params_srv.response.config.ints[i].value;
this->int_params[set_params_srv.response.config.ints[i].name]=set_params_srv.response.config.ints[i].value;
for(i=0;i<set_params_srv.response.config.strs.size();i++)
this->bool_params[set_params_srv.response.config.strs[i].name]=set_params_srv.response.config.strs[i].value;
this->string_params[set_params_srv.response.config.strs[i].name]=set_params_srv.response.config.strs[i].value;
for(i=0;i<set_params_srv.response.config.doubles.size();i++)
this->bool_params[set_params_srv.response.config.doubles[i].name]=set_params_srv.response.config.doubles[i].value;
this->double_params[set_params_srv.response.config.doubles[i].name]=set_params_srv.response.config.doubles[i].value;
this->read_done=true;
}
else
......
......@@ -2,6 +2,7 @@
from distutils.core import setup
from catkin_pkg.python_setup import generate_distutils_setup
d = generate_distutils_setup(
scripts=['src/iri_ros_tools/dyn_params.py'],
packages=['iri_ros_tools'],
package_dir={'': 'src'},
requires=['roslib', 'rospy', 'rosservice']
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment