Skip to content

Conversation

@Garr777
Copy link
Collaborator

@Garr777 Garr777 commented Nov 8, 2022

Сабиров Ришат.
Реализовал все команды MACH3 постпроцессора.
Замечание: при реализации команды AbsMov,вывело ошибку отсутсвия данной команды, поеэтому я ее реализовал и оставил закоменнтированной.

@SprutCAMTech SprutCAMTech self-assigned this Nov 8, 2022
Copy link
Owner

@SprutCAMTech SprutCAMTech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Добрый день. Пока успел только мельком промотреть код, детально не разбирался.
Спасибо за хорошее оформление по правилам.
Сгенерировал УП первым же открытым проектом из дистрибутива (Projects\Examples\Milling\3D\Flat_Land.stc) и там есть отличия от управляющей программы, полученной в старом постпроцессоре. Желательно все-таки добиться
схожести.
image
Можно отключить нумерацию кадров в каждой строке "Nxxxx", чтобы упростить сравнение файлов через VSCode (https://vscode.one/diff-vscode/). В старом постпроцессоре это делается в окне настроек постпроцессора, в новом - можно просто деактивировать регистр номера кадра BlockN.

Вместо команды AbsMov в новом постпроцессоре используется обработчик OnGoto(). Это нужно точно поправить, т.к. это основная команда перемещения.

Замечание по стилю оформления кода. Отступы желательно делать везде одинаковые, стандартные - 4 пробела. Удобочитаемость повышается.

После компиляции C# выдает предупреждения
[{ "resource": "Postprocessor.cs", "code": "CS0168", "message": "Переменная \"stp\" объявлена, но ни разу не использована. }]
Это легко правится.

Вместо запросов информации у пользоваиеля при помощи вслывающих окон в новой системе лучше перенести эти параметры в настройки постпроцессора.
Детально можно посмотреть тут
https://kb.sprutcam.com/docs/Inp.Net/16/en/articles/inpcoreorganization/XMLSettingsWindow.html
https://www.youtube.com/watch?v=Cf8Vcz4Emkg&list=PL0ro0ar_MBbOt-DJJYNdHE0fFODUkFZio&index=12

Пока наверное все. Подробнее разбираться буду чуть позже. Коментарии буду в пул реквест добавлять. Все что описал выше можно уже править.

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

Successfully merging this pull request may close these issues.

3 participants