Skip to content

added sgooway door detector #1707

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

Merged
merged 1 commit into from
Aug 6, 2022
Merged

Conversation

rourke750
Copy link
Contributor

No description provided.

@rourke750
Copy link
Contributor Author

In reference to #1701

@zuckschwerdt
Copy link
Collaborator

Thanks! I'd say this is a "x1527-like OTP encoder" device, but not an actual x1527. The sync pulse is missing (bits=24, but should be bits=25 for x1527).
Can you confirm it works with bits=24 but does not work with bits=25, and if so change the info?

@rourke750
Copy link
Contributor Author

So it does work with 25 bits but I noticed it would send a bunch of requests where at 24 it only gets one. Based on the page it says the ic code is 1527. Is the sync bit important, should I change my config to 25 and see what happens?

@zuckschwerdt
Copy link
Collaborator

I see. It is x1527 then :)
The single code with 24 bits is bad/truncated, the sync is missing. All other codes are proper 25 bits.

You should use bits=25 to avoid problems and try out ,unique(suppress duplicate row output) or ,countonly (suppress detailed row output).

@rourke750
Copy link
Contributor Author

Switched to 25 bits and added unique:
Looks like this
decoder {
name=sgooway,
modulation=OOK_PWM,
short=280,
long=990,
reset=2000,
bits=25,
get=@0:{25}:code,
unique
}
still seeing 20ish messages per broadcast

@zuckschwerdt
Copy link
Collaborator

The get should have a length of {24} only. The last bit is the sync.
The reason for so many messages could be the distance between messages, is it longer than reset?

@rourke750
Copy link
Contributor Author

Set it to 24 bits getting significantly fewer. All messages coming in are within 1 second.

@zuckschwerdt zuckschwerdt force-pushed the master branch 2 times, most recently from 8e31ece to e42ad33 Compare December 26, 2021 11:10
@zuckschwerdt zuckschwerdt merged commit b496305 into merbanan:master Aug 6, 2022
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