-
Notifications
You must be signed in to change notification settings - Fork 5
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
Replace AMBLDC project with AMCBLDC experimental #94
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please, don't remove the old project. Instead, call it amcbldc-old
. I'll get rid of it later.
Also, we need to run a test campaign on the whole wrist before merging, unless this was already done. My understanding of #92 is that you ran tests on a single board. Correct me if I'm wrong.
True that, I'll do both and keep this in draft until the tests on the wrist are done. |
Testing on the robot completed successfully ✅ |
This PR aims to replace the current
AMCBLDC
architectural model with the one present inAMCLBDC_experimental
.The
experimental
project depends on thecommon
project, to reference Simulink models that could be shared between other boards (such as the amc-foc).The simulink models in
common
are:For Each subsystem
#65 and Perform preliminary firmware tests on the AMC-BLDC #84)In the
amcbldc
project the current specialized models are kept:This PR is the result of the tests performed in #92.
Important
The ARM Cortex M CMSIS functions are disabled.