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

Ir issues #799

Open
Yeasyease opened this issue Feb 8, 2025 · 8 comments
Open

Ir issues #799

Yeasyease opened this issue Feb 8, 2025 · 8 comments

Comments

@Yeasyease
Copy link

Custom ir barely works after updating from 1.8.1 -> 1.8.2

Enter custom ir and try to control tv of choice.

  • Device used: Ir transmitter module and m5stickcplus1.1

I expeceted it to on the first attempt to turn off.

I need to send the signal multiple times before the tv actually receives it.

I think the issue could be that Ir signals in general seem to be sent faster than previous version, perhaps make a setting in ir config where you can change the speed it sends the signals to counter this issue.

@Lamnxzp
Copy link
Contributor

Lamnxzp commented Feb 9, 2025

Hey @Yeasyease,
Are you sure you were on Bruce 1.8.1?

Some things changed here: bbdb636 (#635)
Now, unnecessary repeats have been removed (11 signals sent → 1 signal sent).

However, this change also applies to Bruce 1.8.1, so maybe you were actually on Bruce 1.8?

Can you also tell me the protocol of the command you used? (open the IR File (on Bruce or your phone), it's a plain text file, you should see protocol: [something...])

@Yeasyease
Copy link
Author

It says the protocol is Samsung32
The specific file has worked before updating

@Yeasyease
Copy link
Author

And yes before I was using 1.8

@Lamnxzp
Copy link
Contributor

Lamnxzp commented Feb 9, 2025

@Yeasyease
It’s probably because you’re too far from your TV or the IR transmitter isn’t strong enough.

I could add a setting that lets you adjust the number of repeats sent (so it will works the same as before).

@Yeasyease
Copy link
Author

Please do so in a further update

@Yeasyease Yeasyease reopened this Feb 9, 2025
@Yeasyease
Copy link
Author

Okay. It's inconsistent even with the integrated ir transmitter from the m5stickcplus1.1

@Lamnxzp
Copy link
Contributor

Lamnxzp commented Feb 9, 2025

It's working fine on my Samsung TV...

@Yeasyease
Copy link
Author

At least make it a option for compatibility sake.
How long will it take to release the fix?

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

No branches or pull requests

2 participants