-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Decode RAW recordings #1667
Decode RAW recordings #1667
Conversation
Hi. We already have it as a cli command and intentionally decided not to make GUI version of it. Is there any real life use cases for it? |
@skotopes as for me it's cool to have it in gui |
While new protocol support is still being developed, it's a great way to test support for previously recorded codes. I also feel, like @azaitsev, it should be part of the interface. It will also help beginner users to more easily understand and try-out the relationship between raw recordings and their decoded counter parts. |
Could you please point to the respective CLI? |
Checkout my comment in the related issue: #1638 (comment) |
This comment was marked as abuse.
This comment was marked as abuse.
I second this. Some of us cannot use the cli often and, some of us, cannot use the phone app at all (I do not have enough money to buy a new phone and my old one is not supported :/) |
I also very much like this feature. Unlike the CLI command, it creates an output in a format you can save. With the CLI, you'd have to take the text output and manually convert it into the .sub file format. For me this feature is useful where you want to limit the time "in the field", for example when recording doorbells, as you can only trigger the bell twice (record raw and then try if the recording works), and then use this UI to decode the recording or even decode it later once the protocol gets added. If you were to start with non-raw recordings and the protocol was unsupported, you increase the number of recording attempts you have to make. All-in-all for me this is a very nice additition to the FW. |
Hi. Sorry for long delay. We started to split SubGhz application into specialized apps and can accept this PR as external application. Would you like to try to make this feature separate app? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comments in PR thread
I'll move this PR to draft, please un-draft when you complete. |
Any updates on this? |
@CodyTolene at this point the best option to implement whole thing as external application. |
What's new
decode
to the More menu of saved RAW recordingssubghz_file_encoder_worker_get_text_progress
The icon could use a little improvement. Unfortunately I'm not a fluent designer.
assets/icons/SubGhz/Decoding_123x52.png
Verification
More
menuDecode
optionChecklist (For Reviewer)