Yarp: Support yarp::sig::VectorOf<int> in Python

Created on 17 Nov 2020  路  3Comments  路  Source: robotology/yarp

Is your feature request related to a problem? Please describe.

I would like to use yarp::sig::VectorOf<int> in Python. It seems that yarp.i is missing the specialization of yarp::sig::VectorOf for integers.

Additional context

I tried to replicate all the recipes already implemented in yarp.i to support the yarp::sig::VectorOf<double> and I was able to make it work (I only tested in Python though). The changes are in this commit. However, I am not sure whether all these changes are required and sound.

Is it ok if I open a PR?

Bindings FeaEnh Req

Most helpful comment

However, I am not sure whether all these changes are required and sound.

Assessing that is exactly the point of opening a PR, so I would say go forward and let's see the changes in a PR, thanks (even if probably given the state of the CI @drdanz could have other ideas).

All 3 comments

cc @drdanz @diegoferigo

However, I am not sure whether all these changes are required and sound.

Assessing that is exactly the point of opening a PR, so I would say go forward and let's see the changes in a PR, thanks (even if probably given the state of the CI @drdanz could have other ideas).

The CI using github actions is almost working, feel free to open the PR, I will eventually test it locally, or delay it for a couple of days in order to test it properly... Thanks!

Was this page helpful?
0 / 5 - 0 ratings

Related issues

jeljaik picture jeljaik  路  3Comments

traversaro picture traversaro  路  3Comments

diegoferigo picture diegoferigo  路  3Comments

drdanz picture drdanz  路  3Comments

traversaro picture traversaro  路  3Comments