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

minRssi setting per device #168

Closed
mKeRix opened this issue Apr 12, 2020 · 3 comments
Closed

minRssi setting per device #168

mKeRix opened this issue Apr 12, 2020 · 3 comments

Comments

@mKeRix
Copy link
Owner

mKeRix commented Apr 12, 2020

Is your feature request related to a problem? Please describe.
Different device types lead to different RSSI readings at the same distance with Bluetooth Classic. This makes the blanket minRssi setting too generic when using e.g. an iPhone and Apple Watch in the same setup.

Describe the solution you'd like
minRssi should be allowed to be set per device, with a configurable default value for everything that is not specifically configured.

Additional context
Feature request by @lawhazl in #140.

@mKeRix mKeRix closed this as completed in cf3ddc5 Apr 13, 2020
github-actions bot pushed a commit that referenced this issue Apr 13, 2020
# [2.4.0](v2.3.0...v2.4.0) (2020-04-13)

### Bug Fixes

* include distances in API for room presence sensors ([567327d](567327d))
* use Tini for process management in Docker ([932d603](932d603)), closes [#157](#157)

### Features

* simplify log format ([0f90eaa](0f90eaa)), closes [#170](#170)
* **bluetooth-classic:** allow device-specific minRssi values ([cf3ddc5](cf3ddc5)), closes [#168](#168)
* **bluetooth-low-energy:** allow the update frequency to be throttled ([0143309](0143309)), closes [#125](#125)
@github-actions
Copy link

🎉 This issue has been resolved in version 2.4.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@lawhazl
Copy link

lawhazl commented Apr 14, 2020

This is fantastic, I'll get testing this straight away

@readeral
Copy link
Contributor

This is super awesome

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