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

Support for a remote parameter database #22836

Merged
merged 20 commits into from
Mar 11, 2024
Merged

Support for a remote parameter database #22836

merged 20 commits into from
Mar 11, 2024

Conversation

katzfey
Copy link
Contributor

@katzfey katzfey commented Mar 4, 2024

Solved Problem

The voxl2 has a split architecture. PX4 runs on a posix platform and a Qurt platform. The two communicate uorb topics back and forth with the muorb module. But each has it's own parameters database and they need to stay in sync with each other. This PR adds support to keep the 2 parameter databases in sync. The main parameters database running on Linux has file system support while the Qurt one does not. The Linux side is considered the primary and the Qurt side is considered the remote.

@katzfey katzfey requested a review from dagar March 4, 2024 20:40
Copy link
Member

@dagar dagar left a comment

Choose a reason for hiding this comment

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

I think we can live with this for now as long as we're okay dealing with some churn down the line in order to get to a clean parameter server.

@dagar dagar merged commit f4ebfa6 into main Mar 11, 2024
91 of 92 checks passed
@dagar dagar deleted the pr-parameter_remote branch March 11, 2024 17:52
@mrpollo
Copy link
Contributor

mrpollo commented Mar 11, 2024

Happy to see this come in, good work @katzfey 👏

Peize-Liu pushed a commit to Peize-Liu/PX4-Autopilot that referenced this pull request Mar 24, 2024
)

The voxl2 has a split architecture. PX4 runs on a posix platform and a Qurt platform. The two communicate uorb topics back and forth with the muorb module. But each has it's own parameters database and they need to stay in sync with each other. This PR adds support to keep the 2 parameter databases in sync. The main parameters database running on Linux has file system support while the Qurt one does not. The Linux side is considered the primary and the Qurt side is considered the remote.
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

3 participants