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

Feature omniring #70

Open
wants to merge 26 commits into
base: survey-uuids
Choose a base branch
from

Conversation

reyvababtista
Copy link
Collaborator

@reyvababtista reyvababtista commented Nov 14, 2024

This PR adds OmniRing wearable connectivity and listener into Beiwe. More information about OmniRing can be found here: https://www.cse.psu.edu/~mkg31/projects/omniring/

Requirements for merging:

  • Bluetooth permission should only be prompted if a study has bluetooth features enabled.
  • make code copyright statements match your name, I will do the same in files I create or totally rewrite.
  • document what those omniring_headers csv headers are
  • for classes that subclass a system class that is new to the codebase, document what it is, comment something basic about which overridden functions apply to which superclass.
  • show some explaination / origin / attribution for functions that are specific to OmniRing (the bit fiddling code)
  • document bluetooth states that exist so that we know we have coverage, point to docs, stick a date on that documentation line

(cherry picked from commit bca18690ceee64cc19fbd035135a3c3ebfe2cad7)
(cherry picked from commit 8115cd5c9bf0a69dd43c4293a65d429fecf344aa)
(cherry picked from commit e3c5a67d37176fac72082a4a8fcb4cbaf9ff64a7)
(cherry picked from commit 5fdcb72a48459737b5e278c715870de5c630c0af)
(cherry picked from commit 37ac70a3e46c66aaa363c75199d96ab3337b710c)
(cherry picked from commit 306cab56acf4b5056117f07c008f30b9625218b1)
(cherry picked from commit 4566bde75ce79abe58f43ed4e311cbfdf28771b7)
(cherry picked from commit d10391ca6c7fcec35b3d9b652eed2b9a0dc91e92)
(cherry picked from commit c0b607a4adbd9eb6e8a6ed45c8956c68d898d673)
(cherry picked from commit a156c175c9ed153d19d47b969290d624a479c346)
(cherry picked from commit b82c82c31287646cadcb6408c768d461860c798c)
(cherry picked from commit d7b1b387d6a390e63eaafeedd210fc69abc67212)
(cherry picked from commit db30df4705092489b0cfd0a060b69488a26da636)
(cherry picked from commit 9fc39110675295f0e7d3ca7a4f463fc93378ac9f)
(cherry picked from commit 7b29f3b1e642f29e27decf137ae0ac3726d5fdc1)
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.

1 participant