WIP: process_can #89
No reviewers
solid-sinusoid
Labels
No labels
construct
design
doc
integration
programming
research
schematics
test
No milestone
No assignees
2 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 цикл? Сделать ограничение на итерации или сделать ограничения на количество обрабатываемых сообщений...
View command line instructions
Checkout
From your project repository, check out a new branch and test the changes.