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

Performance improvement #77

Open
wbrett opened this issue Mar 30, 2021 · 2 comments
Open

Performance improvement #77

wbrett opened this issue Mar 30, 2021 · 2 comments

Comments

@wbrett
Copy link

wbrett commented Mar 30, 2021

Is your feature request related to a problem? Please describe.
We are seeing very slow response times using the library to write to a NUS serial service tx characteristic. For example, using python with the bleak BLE library we are seeing around 7-10ms response times. For the same request using angular-web-bluetooth library we are seeing on average 300 - 400ms response times. Our application requires many requests to load data from our devices so we would love to see similar performance to the python script.

Describe the solution you'd like
An improvement in the amount of time for the write subscription to return. Here is an example of a request we are making:
this.bluetooth.writeValue$(this.feritConnection.rx, slice)
.subscribe(() => { console.log("This on average takes around 300 - 400ms to return");};

Additional context
Cleaner screenshot of code we are using to write to the NUS serial service tx characteristic:

angularBluetoothWriteRequestCode

Definition of the NUS serial service:
nusSerialSerrvice

@ArctoosR
Copy link

ArctoosR commented Feb 1, 2023

any follow up on this?

@manekinekko
Copy link
Owner

@wbrett Thank you for reporting the performance issue. However, I cannot reproduce the issue because I don't have to the hardware/service. I will leave this question open in case someone can help/suggest a fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants