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

AP_Scripting: fix mavlink_test.lua test #25693

Merged
merged 3 commits into from
Dec 5, 2023

Conversation

peterbarker
Copy link
Contributor

No description provided.

@peterbarker peterbarker force-pushed the pr/scriping-test-nicer branch from 8a17c4c to 2c20d33 Compare December 4, 2023 10:49
@@ -31,7 +31,7 @@ function update()
end
test_named_value = test_named_value + 1.0
-- send named value float
mavlink.send_chan(0, mavlink_msgs.encode("NAMED_VALUE_FLOAT", {time_boot_ms = millis():toint(), name = str_to_bytes("test"), value = test_named_value}))
mavlink.send_chan(0, mavlink_msgs.encode("NAMED_VALUE_FLOAT", {time_boot_ms = millis():toint(), name = "test", value = test_named_value}))
Copy link
Member

Choose a reason for hiding this comment

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

I think you can remove the str_to_bytes function altogether.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@peterbarker peterbarker force-pushed the pr/scriping-test-nicer branch 2 times, most recently from 2294862 to 84395ae Compare December 5, 2023 00:33
@peterbarker peterbarker force-pushed the pr/scriping-test-nicer branch from 84395ae to 7059ba1 Compare December 5, 2023 01:04
@peterbarker peterbarker merged commit 455610a into ArduPilot:master Dec 5, 2023
87 of 88 checks passed
@peterbarker peterbarker deleted the pr/scriping-test-nicer branch December 5, 2023 22:21
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