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

X Axis Flip Still Needed? #3

Open
mdicicco opened this issue Mar 6, 2020 · 2 comments
Open

X Axis Flip Still Needed? #3

mdicicco opened this issue Mar 6, 2020 · 2 comments

Comments

@mdicicco
Copy link

mdicicco commented Mar 6, 2020

Is it still true that ATI sensors don't use the right hand rule for their sensor axes? I have a new Axia80 and it seems to be correct (i.e. it uses a right hand coord system). The raw netft_data seems to match up with the +X and +Y labels on the sensor and the +Z data lines up with a right hand rule. The flip on lines 244/247 of netft_utils seems to be unnecessary.

I did a quick scan of the ATI website and first two sensor drawings in their pdf (the nano17 and then the nano17e) both use a right hand rule as well, although I don't have any other sensors to test on.

I modified my local copy but it could be a good thing to either note or put in a config to disable.

@jbeck28
Copy link

jbeck28 commented Jun 23, 2020

I reached out to an applications engineer with ATI, she told me that all of their sensors follow the right hand rule. So I would think the flip is unnecessary.

@rr-tom-noble
Copy link

rr-tom-noble commented Apr 18, 2023

@AndyZe

We've fallen into the same pitfall due to the x axis inversion, and are having to use our own fork of the repo which does not apply the inversion.

It'd make sense to fix the issue at the source by either removing the inversion (if all are right-handed) or by providing the option to the user (if there's a mix of left-handed and right-handed).

I'm happy to raise a PR, but would appreciate input on which idea is better suited as I'm not familiar with the full range of force sensors ABI offers.

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

3 participants