Merged
Conversation
added 2 commits
February 20, 2026 15:57
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes: #
About
This pull request introduces several improvements and updates across build scripts, documentation, Python client examples, and test code. The most significant changes are enhancements to build system compatibility with Unreal Engine versions 5.2 and 5.7, improved Python client usability, and modernization of test code for better reliability and clarity.
Build system and Unreal Engine compatibility
build.cmdscript to automatically detect Unreal Engine versions 5.2 and 5.7, select the appropriate MSVC toolset, and provide clearer error handling and messaging for standalone and UE-integrated builds.README.mdand Linux development documentation to clarify support for Unreal Engine 5.2 and 5.7, and to emphasize using Unreal's bundled toolchain on Linux instead of system-provided Clang/libc++. [1] [2] [3]Dependency management improvements
CMakeLists.txtto fetch a newer version ofnlohmann-json(v3.11.3) with SHA256 hash validation, and added compiler-specific flags for Assimp to avoid warnings-as-errors. Also setASSIMP_WARNINGS_AS_ERRORS=OFFfor more robust builds. [1] [2] [3]Python client enhancements
keyboard_control.pyfor controlling a drone interactively via keyboard inputs, demonstrating API usage and asynchronous control patterns.Test code modernization
test_hello_drone.pyto use more idiomatic async patterns, improved pose change detection with a timeout, and clarified sensor data validation. Variable naming was updated for clarity and test assertions now wait for actual pose updates before validating movement. [1] [2]Serialization fix
JsonMsgpack::ToJson()incommon_utils.hppto correctly handle msgpack data by passing a pointer range, ensuring reliable deserialization.Credits to @Uginchus
How Has This Been Tested?
In 5.2 and 5.7 win and linux.
Screenshots and videos (if appropriate):