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.

Sign up for GitHub

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

Jump to bottom

pkg/uwb-core: move twr helpers to its own module #18178

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

pkg/uwb-core: move twr helpers to its own module #18178

wants to merge 1 commit into from

Conversation

fjmolinas
Copy link
Contributor

Contribution description

This PR moves the TWR helpers from the example to its own modules. It also adds more tests for the python side. This makes it easier to implement automatic integration tests.

Testing procedure

Follow the application README, it should still apply and work as advertised

Issues/PRs references

Needed by RIOT-OS/Release-Specs#249

@github-actions github-actions bot added Area: build system Area: Build system Area: examples Area: Example Applications Area: Kconfig Area: Kconfig integration Area: pkg Area: External package ports Area: tests Area: tests and testing framework labels Jun 7, 2022
@fjmolinas fjmolinas force-pushed the pr_uwb_core_twr_module branch from e3b9a06 to 3c3719a Compare June 8, 2022 08:15
@fjmolinas fjmolinas added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Jun 8, 2022
@maribu maribu added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Jun 8, 2022
@fjmolinas fjmolinas force-pushed the pr_uwb_core_twr_module branch from 3c3719a to d01ffbc Compare June 9, 2022 09:44
@fjmolinas fjmolinas force-pushed the pr_uwb_core_twr_module branch from d01ffbc to 987d767 Compare June 10, 2022 11:22
@gschorcht gschorcht mentioned this pull request Jun 16, 2022
Merged
@fjmolinas fjmolinas mentioned this pull request Jul 3, 2022
Open
@fjmolinas
Copy link
Contributor Author

Maybe @aabadie can take a look?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leandrolanzieri leandrolanzieri Awaiting requested review from leandrolanzieri leandrolanzieri is a code owner

@jia200x jia200x Awaiting requested review from jia200x jia200x is a code owner

@MrKevinWeiss MrKevinWeiss Awaiting requested review from MrKevinWeiss MrKevinWeiss is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
Area: build system Area: Build system Area: examples Area: Example Applications Area: Kconfig Area: Kconfig integration Area: pkg Area: External package ports Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fjmolinas @maribu