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

Tools: Set SYSID based on instance number in sim_vehicle.py #15758

Closed
wants to merge 1 commit into from

Conversation

araneg
Copy link
Contributor

@araneg araneg commented Nov 9, 2020

Fix for multi-vehicle simulations.

Copy link
Contributor

@khancyr khancyr left a comment

Choose a reason for hiding this comment

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

Good !

Tools/autotest/sim_vehicle.py Outdated Show resolved Hide resolved
@araneg
Copy link
Contributor Author

araneg commented Nov 18, 2020

@tridge
I went ahead and force pushed those commits from the other branch.

@araneg araneg force-pushed the sim_vehicle_sysid branch 2 times, most recently from 656e9f3 to 2617b92 Compare November 18, 2020 17:04
@araneg
Copy link
Contributor Author

araneg commented Nov 23, 2020

@peterbarker
@khancyr
Thoughts?

@tridge
Copy link
Contributor

tridge commented Nov 30, 2020

still fails, with:
image
ArduPlane : /home/tridge/project/UAV/APM/build/sitl/bin/arduplane -S --model plane --speedup 1 --defaults /
home/tridge/project/UAV/APM/Tools/autotest/default_params/plane.parm -I0 --sysid 0

@araneg
Copy link
Contributor Author

araneg commented Nov 30, 2020

Whoops, pushed the wrong branch. I've rebased, should be good now.

@araneg araneg force-pushed the sim_vehicle_sysid branch 3 times, most recently from fa72d31 to 770aea5 Compare December 7, 2020 15:47
Tools/autotest/sim_vehicle.py Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants