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

modbt: GATTCCharacteristic, add read_format() #123

Closed

Conversation

semireg
Copy link

@semireg semireg commented Jan 15, 2018

Add ability to read GATT descriptor Characteristic Presentation Format (0x2904). Returns descriptor value as bytes. Returns None when characteristic is missing the descriptor.

This is more a proof-of-concept because it currently lacks documentation and may not adhere to the standards of this project.

Cross-posted to forum at https://forum.pycom.io/topic/2460/ble-features-non-blocking-connects-char-presentation-format.

caylanlarson and others added 2 commits January 15, 2018 16:55
Add ability to read GATT descriptor Characteristic Presentation Format (0x2904). Returns descriptor value as bytes. Returns None when characteristic missing this descriptor.
@husigeza
Copy link
Contributor

Hello,
I reviewed and tested you implementation, it seems good.
I introduced a little optimization in bt_char_read_format() function.

@husigeza
Copy link
Contributor

husigeza commented Aug 7, 2018

Hello,

Development release 1.19.0.b2 contains an extended functionality with implementing the characteristic.read_descriptor(uuid) function which makes possible to read any descriptor based on the uuid.
Thanks for the idea and contribution!

@husigeza husigeza closed this Aug 7, 2018
peter-pycom pushed a commit that referenced this pull request Jun 10, 2020
Standardized the memory allocations. (Extended from PR 418)
X-Ryl669 pushed a commit to X-Ryl669/pycom-micropython-sigfox that referenced this pull request May 12, 2023
Standardized the memory allocations. (Extended from PR 418)
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.

None yet

3 participants