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

Support for device 0fcf:1009 #59

Closed
jberger opened this Issue May 18, 2013 · 9 comments

Comments

Projects
None yet
2 participants
@jberger
Copy link
Contributor

jberger commented May 18, 2013

My wife's garmin 910xt was working great with this software, then when the unit itself leaked water and was replaced, the replacement came with a new ANT whose dev id is 0fcf:1009. Manually changing the value in ant/fs/manager.py worked, but this is a fragile solution. Could support for both be added somehow? Thanks

BTW as a Perl dev, I would hate to get this message, but I'm not sure how you would want to structure the change to allow for the new dev id. I would be happy to work with you on this, not just drop it in your lap.

@Tigge

This comment has been minimized.

Copy link
Owner

Tigge commented May 20, 2013

Interesting, I'll try to add this on the drivers branch (which is where the main developement go these days). Should be quite trivial to add.

@jberger

This comment has been minimized.

Copy link
Contributor Author

jberger commented May 20, 2013

Good to hear. I'm happy to test if you get a candidate!

@Tigge

This comment has been minimized.

Copy link
Owner

Tigge commented May 20, 2013

Added here: 929fcbc

Should hopefully work

@jberger

This comment has been minimized.

Copy link
Contributor Author

jberger commented May 20, 2013

Hmmm, it seems she must have taken the watch with her to work. I'll test it when she comes home (soon). Are there any actual tests for communication or is just file download sufficient?

@Tigge

This comment has been minimized.

Copy link
Owner

Tigge commented May 20, 2013

Since just changing the usb id worked before: as long as it displays the capabilities string with the usb stick in it should work fine (no need to have the watch there).

@jberger

This comment has been minimized.

Copy link
Contributor Author

jberger commented May 20, 2013

With PR #61 I get

Driver available: [<class ant.base.driver.SerialDriver at 0xb717f8fc>, <class ant.base.driver.USB2Driver at 0xb717fc8c>, <class ant.base.driver.USB3Driver at 0xb718d89c>]
 - Using: ant.base.driver.USB3Driver
Request basic information...
  Capabilities:  array('B', [8, 8, 0, 186, 54, 0, 223, 57])
Starting system...
Key done...
Searching...

which looks correct.

@Tigge Tigge closed this Jun 10, 2013

@jberger

This comment has been minimized.

Copy link
Contributor Author

jberger commented Jun 16, 2013

BTW, this patch is working successfully. Thanks a lot!

@jberger

This comment has been minimized.

Copy link
Contributor Author

jberger commented Sep 7, 2013

Any estimate on when the drivers branch is going to be merged? I can do it manually on my end of course, but I just thought I would ping

@Tigge

This comment has been minimized.

Copy link
Owner

Tigge commented Sep 25, 2013

I'd like to fix issue #47 before merging this branch, but that seems to take some time, which I currently don't have, unfortunately.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.