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

Ookbrute #2354

Merged
merged 8 commits into from
Nov 11, 2024
Merged

Ookbrute #2354

merged 8 commits into from
Nov 11, 2024

Conversation

htotoo
Copy link
Member

@htotoo htotoo commented Nov 11, 2024

OOK brute force ext app.

@gullradriel
Copy link
Member

Wooooow !
Nice one, @htotoo !!!
@zxkmm , may you check this ? :-D

@zxkmm
Copy link
Contributor

zxkmm commented Nov 11, 2024

Nice work!!!
And thanks @gullradriel mentioned me!
It's too complicated so I need to try understanding it firstly, if I can't understand it, I can test and if it's working good, I'll approve.
Also feel free to wait others approve or feel free to merge yourself!

@gullradriel
Copy link
Member

gullradriel commented Nov 11, 2024

Yeah, I want you to test. IIRC you're quite the OOK tester ;-)
I'll also do my job and read the code too.

@zxkmm
Copy link
Contributor

zxkmm commented Nov 11, 2024

I'm hard coding the count to apply on my real 1527 doorbell to test it apply in real world

And after this I will write OK to manual input for NumberField for apps that have large NumberField

image

@htotoo
Copy link
Member Author

htotoo commented Nov 11, 2024

Doing some tests.

@htotoo htotoo merged commit 4641dcb into portapack-mayhem:next Nov 11, 2024
3 checks passed
htotoo added a commit to htotoo/portapack-mayhem that referenced this pull request Nov 12, 2024
htotoo added a commit to htotoo/portapack-mayhem that referenced this pull request Nov 12, 2024
@htotoo htotoo deleted the ookbrute2 branch November 14, 2024 05:19
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.

3 participants