MarsRoverFirmware icon indicating copy to clipboard operation
MarsRoverFirmware copied to clipboard

Create current sensor class

Open younesr1 opened this issue 5 years ago • 8 comments

We need to add a current sensor class to our lib.

younesr1 avatar Sep 27 '20 21:09 younesr1

@a-gratton assign urself to this if you'll take this on pls

younesr1 avatar Sep 27 '20 22:09 younesr1

https://www.melexis.com/-/media/files/documents/datasheets/mlx91205-datasheet-melexis.pdf

younesr1 avatar Sep 27 '20 22:09 younesr1

@younesr1 is this related to the INA-226 in any way?

FK3wong avatar Sep 27 '20 22:09 FK3wong

@FK3wong sort of lol. We're replacing the INAs with this lol

wmmc88 avatar Sep 28 '20 03:09 wmmc88

Are we still going to use the INA-226 for anything then

FK3wong avatar Sep 28 '20 13:09 FK3wong

this is for the current sensors on the arm which are not ina-226. the data sheet is linked above. bms might still be restarted at some point but we dont know yet

younesr1 avatar Sep 28 '20 13:09 younesr1

School ended up being really busy for a while, I am assigning this to myself now.

a-gratton avatar Oct 30 '20 00:10 a-gratton

Just a fyi, we'll need to make sure the driver reads rms current since it's a pwm wave

wmmc88 avatar Feb 26 '21 14:02 wmmc88