Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

yarprobotinterface silently suppresses groups in which one parameters list is not ending with a parenthesis #832

Closed
traversaro opened this issue Jul 19, 2016 · 3 comments

Comments

@traversaro
Copy link
Member

traversaro commented Jul 19, 2016

Example:

  <device name="wholebodydynamics" type="wholebodydynamics">
        <param name="modelFile">model.urdf</param>
        <param name="fixedFrameGravity">(0,0,-9.81)</param>

        <group name="FOO">
            <param name="bar">(1.0,2.0,
                               3.0,4.0</param>
        </group>
  </device>

In this case the FOO group is simply not passed to the open of the device, without any printed warning or error. If the parser encounter an error of this kind, it should fail as soon as possible to simplify debugging of malformed configuration files.

@DanielePucci @fjandrad

@traversaro traversaro changed the title yarpmotorinterface silently suppresses groups in which one parameters list is not ending with a parenthesis yarprobotinterface silently suppresses groups in which one parameters list is not ending with a parenthesis Jul 19, 2016
@DanielePucci
Copy link

-.-

without any printed warning or error

massimoregoli pushed a commit to massimoregoli/yarp that referenced this issue Dec 2, 2016
… balanced and added protection against syntax errors
drdanz pushed a commit to massimoregoli/yarp that referenced this issue Dec 12, 2016
… balanced and added protection against syntax errors
drdanz added a commit that referenced this issue Dec 13, 2016
Fix for issue #832 and further related improvements
@drdanz drdanz added Affects: YARP v2.3.68 This is a known issue affecting YARP v2.3.68 Resolution: Fixed Fixed in: YARP v2.3.70 labels Dec 13, 2016
@Nicogene
Copy link
Member

Nicogene commented May 5, 2017

@traversaro which is the status of this issue? Has been fixed by #995? Can I close it?

@traversaro
Copy link
Member Author

I think so, as far as I remember.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants