Review

[Ivan-Ruzhnikov]

@fuodorov
Вячеслав Васильевич, можете, пожалуйста, посмотреть на то, что сейчас имеется по проекту нашему? Может, есть какие-то комментарии? Насчёт сборки .exe - нужно ли это делать, или без этого пойдёт?

[fuodorov]

@Ivan-Ruzhnikov Привет! Пройдитесь еще раз по коду и исправьте явные косяки, такие как

• Режим MODE_LINEAR?
• while (!Serial.available()) {} - бесконечный цикл?
• int delayBetweenSteps = (100000 / stepsPerSecond) / 2; - переполнение?
• Магические числа
• time.sleep(0.5) - это все на что вы надеетесь?)
• emergency_stop() - просто закрывает порт, а двигатель продолжает движение?
• self.current_position хранится локально - При сбое связи/перезапуске программы потеряете реальное положение

... и тд

И было бы здоров реализовать следующие фичи

• Задавать траектории движения или автоматическое сканирование диапазона
  типа такого

scan(start=0, end=50, step=5, *args)
# Измерение в точках: 0, 5, 10, 15, ..., 50 мм

• График движения в реальном времени
  отображение зависимости position(time) + история последних 10-100 точек
• Журналирование операций

[2025-12-23 14:32:15] Connected to 
[2025-12-23 14:32:20] Set zero at position 15 mm
[2025-12-23 14:32:25] Moving: from 0.000 to 25.000 mm
[2025-12-23 14:32:30] Movement completed successfully

[fuodorov]

экзешник не нужен, нужны инструкции по запуску и указания всех зависимостей

[Ivan-Ruzhnikov]

@fuodorov Здравствуйте! Спасибо, исправлю. Только есть вопрос по поводу MODE_LINEAR и emergency_stop. Нам дали код в теплофизе - прошивку для Arduino. Я в Си и в плюсах не очень сильно понимаю что-то, поэтому через нейронку спросил, что за что отвечает. И как я понял, у установки есть два режима работы - поворота угла и движения по оси, поэтому и добавил вот этот режим. Нужно ли что-то поменять?

И про emergency_stop: в файле .ino (как я понял) этот момент сделан не лучшим образом, то есть нельзя остановить работу движка, пока он не проедет до конца. Поэтому вот такой костыль в коде получился

Ну и про time.sleep тоже чуть-чуть не понял, что имеется ввиду🙃. С остальным все ясно - исправлю и добавлю.

Спасибо за помощь😊

[fuodorov]

@Ivan-Ruzhnikov

• у вас в бекенде есть обращение к MODE_LINEAR, а в инструкциях для ардуино он весь закомментирован - нехорошо.
• с emergency_stop проблема вроде ясна. Но можете оставить так, если не хотите лезть в протокол ардуино. Или вернуть моторчик в начальное положение. Ну или как нибудь сообщите юзеру, что могут быть проблемы. В общем, есть много вариантов более понятного завершения работы)
• с time.sleep много проблем от блокировки выполнения всего приложения до проблем с синхронизацией и состоянием. Если найдете решение лучше, будет здорово.

Надеюсь вы работает в команде)

С остальным все ясно - исправлю и добавлю.

[Ivan-Ruzhnikov]

@fuodorov
Все, понятно. Спасибо большое👍

Да, делаем вместе всё👌