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

Bluetooth: Mesh: alternate location publishing #12005

Merged
merged 1 commit into from
Aug 11, 2023

Conversation

alxelax
Copy link
Contributor

@alxelax alxelax commented Aug 10, 2023

The Location model shall support model publication. When configured for publication, the Generic Location Server shall publish Generic Location Global Status messages if the server knows its global location or shall publish Generic Location Local Status messages if the server knows its local location. If configured for periodic publications, and both global and local locations are known, the Generic Location Server shall alternate between publishing its local and global locations in consecutive publish periods.

@alxelax alxelax requested a review from ludvigsj as a code owner August 10, 2023 11:54
@github-actions github-actions bot added ble mesh Label for ble mesh PRbot. Add this if PR is related to ble mesh and you need to get review. changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Aug 10, 2023
@alxelax alxelax removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Aug 10, 2023
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Aug 10, 2023

Test specification

CI/Jenkins/NRF

  • Integration Platforms

CI/Jenkins/integration

Test Module File based changes Manually selected West overwrite
test-fw-nrfconnect-ble_mesh X

Detailed information of selected test modules

Note: This message is automatically posted and updated by the CI

@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Aug 10, 2023
@alxelax alxelax removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Aug 10, 2023
subsys/bluetooth/mesh/gen_loc_srv.c Outdated Show resolved Hide resolved
@NordicBuilder
Copy link
Contributor

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publishing GitHub Action.

The Location model shall support model publication.
When configured for publication, the Generic Location Server
shall publish Generic Location Global Status messages
if the server knows its global location or shall publish Generic
Location Local Status messages if the server knows its local location.
If configured for periodic publications, and both global
and local locations are known, the Generic Location Server
shall alternate between publishing its local and global locations
in consecutive publish periods.

Signed-off-by: Aleksandr Khromykh <aleksandr.khromykh@nordicsemi.no>
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Aug 10, 2023
@alxelax alxelax removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Aug 10, 2023
@alxelax alxelax requested a review from ludvigsj August 10, 2023 14:48
@akredalen akredalen self-requested a review August 11, 2023 07:44
@rlubos rlubos merged commit cce2a4d into nrfconnect:main Aug 11, 2023
14 checks passed
@alxelax alxelax deleted the alternate_location_pub branch August 11, 2023 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ble mesh Label for ble mesh PRbot. Add this if PR is related to ble mesh and you need to get review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants