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

Disabling of init sequence not working #3

Open
deimi opened this issue Nov 20, 2019 · 5 comments · May be fixed by #4
Open

Disabling of init sequence not working #3

deimi opened this issue Nov 20, 2019 · 5 comments · May be fixed by #4

Comments

@deimi
Copy link

deimi commented Nov 20, 2019

If the checkbox for the init sequence is disabled, the module is still doing the init sequence before sending the binary.

A workaround is to set the init sequence timings to zero even if the checkbox is not selected.

@deimi deimi linked a pull request Nov 22, 2019 that will close this issue
@SergiuToporjinschi
Copy link
Owner

Hi @deimi, I saw that you've done a fork for this. I had to work other projects, and now I want to ask you if your changes are solving your problem. Is true that I did not test it too much without initialization sequence, because I don't use it without it.
If it works for you can I merge back your changes? to test my scenario ?

@deimi
Copy link
Author

deimi commented Jan 17, 2020

Hi!
Yes my changes are working fine for receivers which are not able to handle an init sequence.

Due to lack of receivers which require an init sequence, I have checked the output of the transmitter with an oscilloscope. It looks good to me and should work for your receivers which need an init sequence.

@SergiuToporjinschi
Copy link
Owner

@deimi good if you want you can make a pull request and I will merge it.
is good that you have an oscilloscope :) I've created this node by copying the radio signal using an 433 sniffer and measure the timing for each bit. And because is so hard for me to debug I'm not doing too many changes to it because if something go wrong is a nightmare to debug.

@deimi
Copy link
Author

deimi commented Jan 17, 2020

I already made a pull request for this in november after I verified the changes in my fork.
See #4

@deimi
Copy link
Author

deimi commented Jul 28, 2020

@SergiuToporjinschi status?

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 a pull request may close this issue.

2 participants