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

Replaced test_motor w/ fakeMotionControl #2

Merged
merged 2 commits into from
Jun 14, 2021
Merged

Conversation

pattacini
Copy link
Member

@pattacini pattacini commented Jun 14, 2021

The plugin test_motor has been deprecated.

cc @mfussi66.

@pattacini pattacini requested a review from traversaro June 14, 2021 14:50
@pattacini pattacini self-assigned this Jun 14, 2021
@mfussi66
Copy link

mfussi66 commented Jun 14, 2021

I think it also needs fakeRobotConfig.put("allow-deprecated-devices", 1);, otherwise it quits with an error:

[1623682245.250370] [ERROR] /home/mfussi/iit/yarp/src/libYARP_dev/src/yarp/dev/PolyDriver.cpp:321 bool yarp::dev::PolyDriver::coreOpen(yarp::os::Searchable&) (0x000d7051) |yarp.dev.PolyDriver| Device "fakeMotor" is deprecated. Pass the "allow-deprecated-devices" option in the configuration if you want to open it anyway.

I just tested the program with the renamed device and the above config option, and everything seems to work fine.

@pattacini
Copy link
Member Author

Hi @mfussi66,
Actually, this PR proposes to use the new device: so it should be all good.

@traversaro
Copy link

Interestingly, also fakeMotor is deprecated robotology/yarp#2478 . So if fakeMotionControl is working fine on YARP 3.4, I would just go for fakeMotionControl directly.

@pattacini
Copy link
Member Author

Good catch @traversaro
Let me have a loot at it.

@pattacini
Copy link
Member Author

pattacini commented Jun 14, 2021

fakeMotionControl is not enabled by default in the robotology-superbuild and this turns to be problematic for our infrastructure.

I've checked that in https://github.com/orgs/icub-training/packages/container/sandbox/2890684?tag=v2021.05.

I believe we should make it available.

@pattacini pattacini changed the title Replaced test_motor w/ fakeMotor Replaced test_motor w/ fakeMotionControl Jun 14, 2021
@pattacini pattacini marked this pull request as draft June 14, 2021 15:54
@pattacini
Copy link
Member Author

Switched to the use of fakeMotionControl device.
To be tested.

@pattacini
Copy link
Member Author

Everything's fine 👍🏻
I've tested it with the latest vvv-school image on Gitpod.

@pattacini pattacini merged commit 3945293 into master Jun 14, 2021
@pattacini pattacini deleted the pattacini-patch-1 branch June 14, 2021 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants