Skip to content

GcodeVisualizer: added support specific 5 axis machine#1385

Merged
ABSitf merged 3 commits intomasterfrom
i2198_4_axis_machine
Jul 5, 2023
Merged

GcodeVisualizer: added support specific 5 axis machine#1385
ABSitf merged 3 commits intomasterfrom
i2198_4_axis_machine

Conversation

@ABSitf
Copy link
Contributor

@ABSitf ABSitf commented Jul 3, 2023

No description provided.

@ABSitf ABSitf changed the title added support 4 axis machine GcodeVisualizer: added support 4 axis machine Jul 3, 2023
@ABSitf ABSitf changed the title GcodeVisualizer: added support 4 axis machine GcodeVisualizer: added support 5 axis machine Jul 5, 2023
@ABSitf ABSitf changed the title GcodeVisualizer: added support 5 axis machine GcodeVisualizer: added support specific 5 axis machine Jul 5, 2023
// process all commands from one line g-code source and generate corresponding move action
MRMESH_API MoveAction processLine( const std::string_view& line );

//
Copy link
Contributor

Choose a reason for hiding this comment

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

empty comment

Comment on lines +89 to +91
// a, b, c
BaseAction3f rotateTool_();

Copy link
Contributor

Choose a reason for hiding this comment

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

mb give more meaningfull comment

struct MoveAction
{
BaseAction3f action;
std::vector<Vector3f> toolDirection; // tool direction for each point action.path
Copy link
Contributor

Choose a reason for hiding this comment

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

what is this for?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

removed. will using in next PR for show tool direction in GcodeVisualizer

Vector3<bool> inputCoordsReaded_;
std::optional<float> radius_;
std::optional<Vector3f> arcCenter_;
std::optional<Vector3f> inputRotation_;
Copy link
Contributor

Choose a reason for hiding this comment

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

what is this for?

@ABSitf ABSitf merged commit c110e73 into master Jul 5, 2023
@ABSitf ABSitf deleted the i2198_4_axis_machine branch July 5, 2023 16:24
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.

2 participants