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

Hamilton8415/ehpto switchlossfix #169

Merged
merged 55 commits into from
Dec 8, 2023
Merged

Conversation

hamilton8415
Copy link
Collaborator

This merge brings changes to the electro-hydraulic PTO plugin into main and includes a number of changes to improve the match between the plugins behavior and the bench-test measurements

andermi and others added 30 commits June 26, 2023 11:25
Signed-off-by: Michael Anderson <[email protected]>
… with non-pointer in ECM

Signed-off-by: Michael Anderson <[email protected]>
Signed-off-by: Michael Anderson <[email protected]>
Signed-off-by: Michael Anderson <[email protected]>
Signed-off-by: Michael Anderson <[email protected]>
Signed-off-by: Michael Anderson <[email protected]>
Signed-off-by: Michael Anderson <[email protected]>
Signed-off-by: Michael Anderson <[email protected]>
@andermi andermi self-requested a review December 6, 2023 20:50
@andermi
Copy link
Collaborator

andermi commented Dec 6, 2023

ros2 launch buoy_tests experiment_comparison_select.launch.py manual:=True test_inputdata_exp:='2022.12.21T15.44.24_VentedSpring.exp' test_inputdata_tst:='2022.12.21T15.44.24_VentedSpring.tst'

Signed-off-by: Michael Anderson <[email protected]>
@andermi
Copy link
Collaborator

andermi commented Dec 7, 2023

I didn't like how you have to specify both exp and tst... so, now I just have you enter the basename once and it will do the .exp and .tst for you

@andermi
Copy link
Collaborator

andermi commented Dec 7, 2023

oops... I broke something stupid with the CI version of experiment_comparison. I'll fix it.

@andermi
Copy link
Collaborator

andermi commented Dec 7, 2023

I think I found a much easier way to write the args... 🤦‍♂️

@hamilton8415
Copy link
Collaborator Author

hamilton8415 commented Dec 7, 2023

I never doubted you, I'll give it a run-through whenever it's set.

@andermi
Copy link
Collaborator

andermi commented Dec 7, 2023

I still need to implement it. I'll let you know when it's done :)

@andermi
Copy link
Collaborator

andermi commented Dec 7, 2023

ros2 launch buoy_tests experiment_comparison_select.launch.py manual:=True test_inputdata:='2022.12.21T15.44.24_VentedSpring'

or,

ros2 launch buoy_tests experiment_comparison_select.launch.py manual:=True test_inputdata:='2022.12.21T15.44.24_VentedSpring.exp'

or,

ros2 launch buoy_tests experiment_comparison_select.launch.py manual:=True test_inputdata:='/path/to/2022.12.21T15.44.24_VentedSpring.exp'

it will always look in buoy_tests/test_inputdata/ for the name of the test you specified... it will ignore your path. and it will also ignore the .exp and .tst file extensions

@andermi
Copy link
Collaborator

andermi commented Dec 8, 2023

fixed the absement tests and added some comments for what I did in the _select launch file... The only test not passing is the experiment_comparison now. Need to approve the .tst I think

Copy link
Collaborator

@andermi andermi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@hamilton8415 hamilton8415 merged commit e8513bf into main Dec 8, 2023
2 checks passed
@hamilton8415 hamilton8415 deleted the hamilton8415/ehpto_switchlossfix branch December 8, 2023 21:56
@andermi
Copy link
Collaborator

andermi commented Dec 8, 2023

nice job remembering the CI hehe

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.

2 participants