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

RollEaseAcmedia pulse time refinement #3055

Closed
wants to merge 4 commits into from
Closed

Conversation

bdnstn
Copy link
Contributor

@bdnstn bdnstn commented Sep 19, 2024

Refine the pulse timing by exploring the limits of pulse timings that work and choosing the mid points. Tested an verified against 10 different blind motors on 2 different address and found improved (now 100%) reliability.

@zuckschwerdt
Copy link
Collaborator

Appreciated, thanks! Can you document your findings about the timings ranges?

@bdnstn
Copy link
Contributor Author

bdnstn commented Sep 19, 2024

Sure, in the conf file?
I built a transmitter using Liligo LoRa ESP32 to operate my blinds. I found the original timings were failing about 3% of operations. By experimentation, I found that the upper and lower limits of short pulses for successful operation were 460 and 350 us respectively. In all test I just made the long pulse timing 2*short pulse.
I then chose the mid-point between upper and lower limits (460+350)/2=405us for short and 2*405=810us for long. Subsequent testing of transitions with these timings has been 100% reliable over about 100 cycles.

@zuckschwerdt
Copy link
Collaborator

Yes, a line of something like "short pulses have been found to work between 350 to 460 us, with long pulses twice that."

Note that for the simple case (no tolerance set) the slicer determines short/long at the mid-point between short and long, i.e. for s=352, l=725 it's 538.5 µs and for the new s=405, l=810 it's 607.5 µs. I'd say that both timings should work the same for the decoder, assuming there are no pulses in that mid range of 500-600 µs.

@bdnstn
Copy link
Contributor Author

bdnstn commented Sep 19, 2024

Sorry, it looks like the new conf I have created for OMA controller has got muddled with the PR for the amendment to the rolleasacmedia.conf. I'm a NOOB with github :-(

@zuckschwerdt
Copy link
Collaborator

I'll see if I can fix that with a manual merge.

@zuckschwerdt
Copy link
Collaborator

manually merged, also changed the line endings from DOS (CR+LF) to UNIX (LF).

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.

2 participants