kylemcdonald / ofxConnexion

Wraps 3dConnexionClient for openFrameworks on OSX
Other
13 stars 7 forks source link

Example broken #2

Open laserpilot opened 8 years ago

laserpilot commented 8 years ago
Example line 40:

Needs to be changed from: ofTranslate( data.translation[0], data.translation[2], data.translation[1]);

to

ofTranslate( data.position[0], data.position[2], data.position[1]);

moebiussurfing commented 8 years ago

thanks, I can confirm it's working fine on 10.10 and 0.9.0 master branch. I had to recreate a clean project with the above changes. The included one didn't worked here.

kylemcdonald commented 8 years ago

if either of you get a chance to send a PR it would be appreciated!

thanks :)

moebiussurfing commented 8 years ago

Hey @kylemcdonald, I would like to do it! But sorry, I am not sure how to do the pull request, I have the project in another different folder (/apps/myApps)... Anyway I uploaded the example here: https://db.tt/uqH8dAHJ (I added a little overlay with axes debug on screen.) captura de pantalla 2015-11-05 a las 16 33 37

moebiussurfing commented 8 years ago

btw, do you know if this addon should work too on Windows (Intel) and maybe Raspbian Raspberry? On the official 3dConnexion website there is SDK for all platforms, but Linux it's Intel only, not ARM.

kylemcdonald commented 8 years ago

@moebiussurfing i don't know which platforms this works for. i've used the connexion on OSX and windows, never linux.

if you have a moment to make the PR, you will need to:

  1. fork this repo
  2. checkout your copy of the repo
  3. make the changes, do git add . and git commit -m "updated example" and git push origin master
  4. make a PR on the main page of ofxConnexion on github
moebiussurfing commented 8 years ago

@kylemcdonald : done. I hope is working fine!

moebiussurfing commented 8 years ago

@kylemcdonald . I think now is ok. I deleted the 3dconnexion sdk file.