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

Request for CAN improvements #61

Open
owhite opened this issue Jun 29, 2023 · 0 comments
Open

Request for CAN improvements #61

owhite opened this issue Jun 29, 2023 · 0 comments

Comments

@owhite
Copy link
Collaborator

owhite commented Jun 29, 2023

See: https://github.com/davidmolony/MESC_Firmware/blob/FW_ADC_sampling/MESC_RTOS/MESC/MESCinterface.c

starting around line 493

At present when we perform data logging, we export all these values:

  1. mtr[0].Conv.Vbus , "vbus"
  2. mtr[0].FOC.eHz , "ehz"
  3. mtr[0].Conv.MOSu_T , "TMOS"
  4. mtr[0].Conv.Motor_T , "TMOT"
  5. mtr[0].FOC.Idq_req.q , "iqreq"
  6. mtr[0].FOC.Idq_smoothed.d , "id"
  7. mtr[0].FOC.Idq_smoothed.q , "iq"
  8. mtr[0].Raw.ADC_in_ext1 , "adc1"
  9. MESC_errors , "error"
  10. mtr[0].FOC.Vdq.q , "Vq"
  11. mtr[0].FOC.Vdq.d , "Vd"

I believe only 1-5 are available through CAN, could you send the additional variables? An slight alternative is to basically send the same content as what is transferred with "status json".

Please consider creation of CAN commands to:

  • reset MESC_errors to zero without power-cycling the board
  • control the kill switch function (possibly independent of the kill switch tied to hardware).
  • change state of the LED
  • set "slambrake"
  • store odometer readings
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

1 participant