OpenRoberta / openroberta-lab

The programming environment »Open Roberta Lab« by Fraunhofer IAIS enables children and adolescents to program robots. A variety of different programming blocks are provided to program motors and sensors of the robot. Open Roberta Lab uses an approach of graphical programming so that beginners can seamlessly start coding. As a cloud-based application, the platform can be used without prior installation of specific software but runs in any popular browser, independent of operating system and device.
Apache License 2.0
122 stars 118 forks source link

EV3: faulty simulation with non-standard robot configuration #1657

Open charlie-chaperon opened 2 months ago

charlie-chaperon commented 2 months ago

Describe the bug When using a custom robot configuration by reversing left and right for motors B and C, the simulation does not work properly. When simulating a line follower with a standard configuration, it works well. Reversing Left and right makes the simulated robot drive in a straight line and not reacting to any input from the light sensor. However the same program just works as expected when uploaded to a EV3.

standard robot configuration standard config

custom robot configuration with left and right reversed custom config

Test program used: line following. line-follow program

Steps to reproduce the behavior:

  1. write a simple line follow program for the EV3 c4ev3 (see picture)

  2. use standard robot configuration

  3. simulate program - everything should work

  4. switch sides of the motors from left/right (see picture)

  5. simulate the new program - the robot drives in a straight line and does not react to sensor input

Expected behavior The custom configuration should work like the standard configuration, just with the robot turning to the right instead of the left when not finding a line.

Device information

Additional context The programs do work on a real EV3, just the simulation seems bugged.

bjost2s commented 1 week ago

Thanks for this issue, your enhancement makes sense, we will considerate this in a future version. Currently there is no time to solve it soon.

bjost2s commented 6 days ago

Solution should solve #1507, too