WIP: process_can #89
No reviewers
solid-sinusoid
Labels
No labels
construct
design
doc
integration
programming
research
schematics
test
No milestone
No assignees
3 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
robossembler/servo!89
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "process_can"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
@ -70,0 +70,4 @@void send_torque() {float i_q = motor.current.q; // Q-axis current (A)float torque = 100 * i_q; // Torque calculationЭто откуда такая формула? Если функция пока не дописана пометь ее
// TODO: something
Потом легко в коде ориентироваться
А можно пометить как
// TODO(#45): something
Типа сразу номер issue указываешь, если такой issue есть
@ -0,0 +89,4 @@# 4. Проверка установки нового угла (интеграционно)# ... (может требовать дополнительной проверки на устройстве)Можно добавить тест на изменение режима управленя и отправку скорости. 2 раза подряд угол и 2 раза подряд скорость
@ -98,9 +99,13 @@ MotorControlInputs motor_control_inputs;void loop() {__enable_irq();А без этого не обойтись? Обязательно на каждом цикле включать прерывания? Это можно вынести в инициализацию? Если на каждом цикле включать прерывания, то это ведь может привести к неопределенному поведению?
@ -104,2 +107,4 @@// Process incoming CAN messageswhile (Can.read(msg)) {Может, чтобы не было потенциальных залипаний цикла надо ограничить while цикл? Сделать ограничение на итерации или сделать ограничения на количество обрабатываемых сообщений...
WIP: process_canto process_canprocess_canto WIP: process_canWIP: process_canto process_canprocess_canto WIP: process_canВетку закрываем, так как переходим CubeMX IDE
Pull request closed