Skip to content

add local time gatt characteristic#108

Open
cybuzuma wants to merge 1 commit into
theironrobin:mainfrom
cybuzuma:gatt_localtime
Open

add local time gatt characteristic#108
cybuzuma wants to merge 1 commit into
theironrobin:mainfrom
cybuzuma:gatt_localtime

Conversation

@cybuzuma

Copy link
Copy Markdown

Motivation

InfiniTime gains the Local Time GATT Characteristic in this PR. This characteristic sends information about the current timezone to the watch and thus allows it to calculate the current UTC.

Changes

  • added a function that builds the bytes object for the characteristic. As I do not know of any "standard" way to get information whether DST is in effect in python, the code just always sends the information as DST = 0 (no dst).
  • added code to send local time information if the corresponding characteristic is discovered on the watch
  • corrected the Current Time data to use explicit little endian encoding

@theironrobin

Copy link
Copy Markdown
Owner

@cybuzuma do you still want this pulled in? i'm a bit out of the loop on this

@cybuzuma

Copy link
Copy Markdown
Author

I still do.

While the feature is not yet merged in InfiniTime, I am confident it will at some point.
Having as many companion apps as possible support this would be great for users, especially as siglo is listed as the recommended application on linux, it would be great if siglo supported this feature.

@JF002

JF002 commented Nov 11, 2022

Copy link
Copy Markdown

I've just merged this feature in InfiniTime! It'll be available in version 1.12 ;)

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.

3 participants