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

Fix for disconnecting 1473 and K4W devices. Closes #340 #361

Closed
wants to merge 1 commit into from

Conversation

ofTheo
Copy link
Contributor

@ofTheo ofTheo commented Jan 23, 2014

This fixes a known issue for OS X users where the device re-enumerates within a small time period and causes a freeze.

This PR detects 1473 devices and K4W devices and sets the LED state to solid red.
It seems that in newer 1473 devices and 1473 devices with firmware updated by an Xbox the device will re-enumerate unless the LED is set.

The code is fairly compact, but it needs testing on a K4W device.
@wizgrav - would be great if you could pull and confirm if this sets K4W LED to red as well.

Most of this code is adapted from Drew Fisher's ( @zarvox ) proof of concept tilt code.
Note: it doesn't require the audio firmware to be flashed.

Closes #340 and addresses #316.

Signed-off-by: Theodore Watson theo@openframeworks.cc (ofTheo)

…up. This seems to fix a known issue for OS X users where the device renumerates within a small time period and causes a freeze. Closes OpenKinect#340 and addresses OpenKinect#316. Signed-off-by: Theodore Watson <theo@openframeworks.cc> (ofTheo)
@wizgrav
Copy link
Contributor

wizgrav commented Jan 24, 2014

awesome, I'll have a chance to test a k4w in the weekend. would xtion support also make sense?

@ofTheo
Copy link
Contributor Author

ofTheo commented Jan 25, 2014

I sort of feel for now that sticking with the Kinect devices is a better way to go.
Once we have full parity across k4w, 1473 and 1414 - then it could be a good time to look at expanding.

But getting that parity first would be amazing! :)
I also have a K4W device coming in a couple of days.

I got @zarvox's Tilt and LED for 1473 working really well today and integrated into libfreenect with audio firmware loading from memory:
openframeworks/openFrameworks#2778

Once I have the K4W device I'll test for compatibility and submit a PR to this repo.

It seems really close!
This PR though is really needed for OS X and 1473.

@ofTheo
Copy link
Contributor Author

ofTheo commented Jan 28, 2014

@wizgrav - I got a K4W device and had to do a few fixes but now its working great!

I made a more comprehensive PR ( #362 ) which also allows for the motor/led/accel control on K4W and better checking for device type.

That PR also has the keep_alive code which fixes #340 and #316.
So I am going to close this PR and encourage @zarvox to merge #362 :)

If you are able to test #362 on Win or Nix it would be most appreciated.
Thanks!!
Theo

@ofTheo ofTheo closed this Jan 28, 2014
@piedar piedar removed this from the v0.3.1 milestone Feb 3, 2014
@tufekoa
Copy link

tufekoa commented May 20, 2014

Hi of Theo, Thanks for your work on this issue I just got the Kinect 1473 with the same problem. I was just wondering how to use your code,Where in libfreenect do we have to make the changes. I'm sorry but I am really a bit new to this but am on it night and day. Cheers!

@collinalexbell
Copy link

bump tufekoa, I am having the same troubles

@tufekoa
Copy link

tufekoa commented Dec 26, 2014

I suggest you get a 1414 model. There are reburbished models for sale online. I spent a lot of time on this in pure waste. Cheers

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.

Support for model 1473 on OS X
5 participants