New frequency analyzer#1557
New frequency analyzer#1557ClusterM wants to merge 3 commits intoflipperdevices:devfrom ClusterM:dev
Conversation
ClusterM
requested review from
DrZlo13,
Skorpionm,
hedger and
skotopes
as code owners
|
The sensitivity was intentionally set low by design, to filter out noise. Did you just increase it, or is there some way to change it now? |
|
@ClusterM works fine 👍 awesome PR. would it be possible to add "up" to toggle sound on/off? Sometimes you don't want to hear it 😉 |
It's possible but I'm not sure how to show it in GUI. |
You can change trigger level now. It affects sound and "last" value. |
Fixed, it changes with trigger level now. |
…ices#1557 Signed-off-by: bdryanovski <bozhidar.dryanovski@theopentag.com>
| if((rssi == 0.f) && (instance->locked)) { | ||
| if(instance->callback) { | ||
| instance->callback(SubGhzCustomEventSceneAnalyzerUnlock, instance->context); | ||
| notification_message(instance->notifications, &sequence_hw_blink); |
There was a problem hiding this comment.
@ClusterM I'm not sure, that's a good idea to blink while scanning. It was convenient when Flipper blinked only when it senses a signal. Now it's harder to detect the moment when I need to pay attention to the screen. I'd rather disable constant blinking both here and in Read/Read RAW mode, because it may draw unwanted attention when using in public spaces, and restore the subghz_scene_frequency_analyzer_on_event function.
There was a problem hiding this comment.
There was a problem hiding this comment.
And my PR with Read/Read RAW flashing disabled: #1745
|
Please talk with @Skorpionm regarding this PR. |
7 participants
Reopened pull request.
What's new
New frequency analyzer view:
Verification
Checklist (For Reviewer)