search

simplefoc / Arduino-FOC-drivers

Drivers and support code for SimpleFOC
https://docs.simplefoc.com/drivers_library
MIT License
146 stars 63 forks source link

Adding support for MA735 #45

Closed techy-robot closed 1 month ago

techy-robot commented 1 month ago

This is largely based off of the code for MA730, with some additional functions to support the variable resolution and speed (Tied to the Filter Window register). The MA735 is probably one of the cheapest Mag Alpha sensors out there, and looks like a really nice option from my testing. Resolution is inversely proportional to reading speed, ranging from 9-13 bit resolution and 64uS to 16 ms update times.

runger1101001 commented 1 month ago

Thank you so much for contributing this!

I don't have this sensor to test myself, but am happy to merge it based on your successful tests.

I have changed the PR to be against the dev branch of the library, we only accept PRs to dev.

Just running the test compiles, and assuming it compiles for all our platforms I will merge it.

Thanks again!

techy-robot commented 1 month ago

Interesting the esp32 platform failed to compile. It seems unrelated to my code though.

runger1101001 commented 1 month ago

Yes, that's not a problem - the esp32 upgrade is already on the dev branch of SimpleFOC so this error will be fixed with the next SimpleFOC release. Merging now.