search

RobotML / RobotML-SDK

RobotML-SDK is an implementation of the Robot Modeling language. It is based upon Papyrus modeling tool (http://www.papyrus.org) which is itself based upon Eclipse framework (http://www.eclipse.org).
2 stars 1 forks source link

Morse generation: 'sensor' fields of different sensor classes should have different names #128

Open Farges opened 11 years ago

Farges commented 11 years ago

The use of the word 'sensor':

        self.sensor = VideoCamera()

in CameraMain_002 together with the same word:

        self.sensor = GPS()

in GPS_001 is very confusing for morse because the first one leads to robotRMax.sensor and the second one to robotRMax.sensor.001. From there morse is unable to find the correct references. Substituting the word in GPS_001:

        self.pasLeMemeNomQuePourCamera = GPS()

        self.pasLeMemeNomQuePourCamera.translate(x=0, y=0, z=0)

        self.pasLeMemeNomQuePourCamera.rotate(x=0)

        self.pasLeMemeNomQuePourCamera.frequency(0.0)
        self.pasLeMemeNomQuePourCamera.add_stream('ros')
        myRobot.append(self.pasLeMemeNomQuePourCamera)

leads to a code without error. :-)

SergeStinckwich commented 11 years ago

Are you sure about that ? Normally it should work.

Farges commented 11 years ago

On Tue, 23 Jul 2013 07:20:27 -0700, Serge Stinckwich wrote:

Are you sure about that ? Normally it should work. I am sure. May be it is a Morse issue...

Reply to this email directly or view it on GitHub [1].

Links:

[1] https://github.com/RobotML/RobotML-SDK/issues/128#issuecomment-21417145

Farges commented 11 years ago

I posted an issue for morse : https://github.com/morse-simulator/morse/issues/434

Farges commented 11 years ago

There are two answers to the issue posted in morse. Either generation style should be changed (as suggested by Severin), either we wait for morse 1.1beta (as suggested by Arnaud).

SergeStinckwich commented 11 years ago

Ok thank you Jean-Loup. Can you link the answers here ?

Farges commented 11 years ago

Severin: https://github.com/morse-simulator/morse/issues/434#issuecomment-24139058 and Arnaud: https://github.com/morse-simulator/morse/issues/434#issuecomment-24140163