search

Rat-OS / RatOS-configuration

The RatOS modular klipper configuration
140 stars 131 forks source link

ADXL345 axis mapping #102

Closed GP3D84 closed 1 year ago

GP3D84 commented 1 year ago

Hi, according to official klipper documentation the accelerometer must be installed respecting correct axis Screenshot 2023-01-30 alle 20 24 04 On classic Eva3 mounting, the adxl axis should be remapped by : axes_map: x, y, z (default) to axes_map: x, z, y in RatOS. In any case is better to warning user about alternative adxl mount

@miklschmidt what do you think?

miklschmidt commented 1 year ago

AFAIK this does little more than make sure your graph has the correct legends. There are many, many ways to mount ADXL345's currently, and this is something the user can easily change on their own. I don't think the work and complexity in making sure the axes are mapped correctly in all use cases is worth it.

GP3D84 commented 1 year ago

The legends is always correct despite the incorrect mapping. What changes is the correct reading. Not only the klipper documentation say to respect axis vs adxl mount but in other applications of adxl as well. Anyway, if you don't think it's important, i will close this issue Thank you anyway for all the work you've done

miklschmidt commented 1 year ago

The legends is always correct despite the incorrect mapping.

@GP3D84 What i mean is that it'll show Y vibrations for X etc, depending on the orientation (thus wrong labelling).

Not only the klipper documentation say to respect axis vs adxl mount

Can you point me to where it says that?