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

multi-channel collision #33

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

casang
Copy link

@casang casang commented Jul 3, 2020

Hi @Loghorn,

When I use Garmin Stick to connect to a Garmin Speed and Cadence Sensor, I am receiving "Unhandled Event" error regularly in ant.js file. I debugged the code and the problem is that the event EVENT_CHANNEL_COLLISION is not handled properly.

Collision is not an error on multi-channel networks, as devices uses only one antena.

So I just add the code below to handle EVENT_CHANNEL_COLLISION properly:

case Constants.EVENT_CHANNEL_COLLISION:
return true; // collision is not an error on multi-channel networks

I really appreciate if you can merge these changes.

Find attached ANT AN15 specs: https://www.thisisant.com/assets/resources/Application%20Notes/ANT_AN15_Multi_Channel_Design_Considerations.pdf

Collision is not an error on multi-channel networks
@AdrienFery
Copy link

Hi @Loghorn,

Is there a particular reason why this PR is not merged? I have the same issue.

Thanks,

@bierzorutas
Copy link

+1

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