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

Need to test for old hrpsys-base #302

Open
snozawa opened this issue Dec 26, 2014 · 2 comments
Open

Need to test for old hrpsys-base #302

snozawa opened this issue Dec 26, 2014 · 2 comments

Comments

@snozawa
Copy link
Contributor

snozawa commented Dec 26, 2014

Currently hrpsys-base tests rtmros_hironx with old hrpsys (315.1.9).

Because of recent rtmros_hiornx commit such as [1], hrpsys-base test fails.
For example[2], in hironx_ros_bridge/test/test_hironx_ik.py uses getReferenceRPY with three arguments, but this argument for getReferenceRPY should be two in hrpsys-base 315.1.9.

[1] ff4df59
[2] fkanehiro/hrpsys-base#433

Therefore, we need to add test for rtmros_hionx + old hrpsys (315.1.9).

@k-okada
Copy link
Member

k-okada commented Dec 27, 2014

Currently hrpsys-base tests rtmros_hironx with old hrpsys (315.1.9).
Therefore, we need to add test for rtmros_hionx + old hrpsys (315.1.9).

( Currently A,,, There for we need A )

what does this mean? Any mistake?

@130s 130s removed their assignment Dec 30, 2014
@snozawa
Copy link
Contributor Author

snozawa commented Dec 31, 2014

Sorry, I forgot to reply this...

Currently hrpsys-base tests rtmros_hironx with old hrpsys (315.1.9).
Therefore, we need to add test for rtmros_hionx + old hrpsys (315.1.9).

hrpsys-base has tests for latest rtmros_hironx + old hrpsys-baes (315.1.9).
rtmros_hironx does not have tests for latest rtmros_hironx + old hrpsys-baes (315.1.9),
so we need to add these tests to rtmros_hironx.

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

No branches or pull requests

3 participants