Merge pull request #19976 from clsergent/patch-1
CAM: Fix an error in CAM/Path/Post/UtilsParse.py
This commit is contained in:
@@ -88,7 +88,7 @@ def check_for_drill_translate(
|
||||
+ format_command_line(values, command_line)
|
||||
+ values["COMMAND_SPACE"],
|
||||
)
|
||||
gcode += f"{linenumber(values)}{comment}{nl}"
|
||||
gcode.append(f"{linenumber(values)}{comment}{nl}")
|
||||
# wrap this block to ensure that the value of values["MOTION_MODE"]
|
||||
# is restored in case of error
|
||||
try:
|
||||
@@ -121,7 +121,7 @@ def check_for_machine_specific_commands(values: Values, gcode: Gcode, command: s
|
||||
if m:
|
||||
raw_command = m.group(1)
|
||||
# pass literally to the controller
|
||||
gcode += f"{linenumber(values)}{raw_command}{nl}"
|
||||
gcode.append(f"{linenumber(values)}{raw_command}{nl}")
|
||||
|
||||
|
||||
def check_for_spindle_wait(
|
||||
@@ -132,9 +132,9 @@ def check_for_spindle_wait(
|
||||
nl: str = "\n"
|
||||
|
||||
if values["SPINDLE_WAIT"] > 0 and command in ("M3", "M03", "M4", "M04"):
|
||||
gcode += f"{linenumber(values)}{format_command_line(values, command_line)}{nl}"
|
||||
gcode.append(f"{linenumber(values)}{format_command_line(values, command_line)}{nl}")
|
||||
cmd = format_command_line(values, ["G4", f'P{values["SPINDLE_WAIT"]}'])
|
||||
gcode += f"{linenumber(values)}{cmd}{nl}"
|
||||
gcode.append(f"{linenumber(values)}{cmd}{nl}")
|
||||
|
||||
|
||||
def check_for_suppressed_commands(
|
||||
@@ -153,7 +153,7 @@ def check_for_suppressed_commands(
|
||||
+ format_command_line(values, command_line)
|
||||
+ values["COMMAND_SPACE"],
|
||||
)
|
||||
gcode += f"{linenumber(values)}{comment}{nl}"
|
||||
gcode.append(f"{linenumber(values)}{comment}{nl}")
|
||||
# remove the command
|
||||
return True
|
||||
return False
|
||||
@@ -165,7 +165,7 @@ def check_for_tlo(values: Values, gcode: Gcode, command: str, params: PathParame
|
||||
|
||||
if command in ("M6", "M06") and values["USE_TLO"]:
|
||||
cmd = format_command_line(values, ["G43", f'H{str(int(params["T"]))}'])
|
||||
gcode += f"{linenumber(values)}{cmd}{nl}"
|
||||
gcode.append(f"{linenumber(values)}{cmd}{nl}")
|
||||
|
||||
|
||||
def check_for_tool_change(
|
||||
@@ -177,13 +177,13 @@ def check_for_tool_change(
|
||||
if command in ("M6", "M06"):
|
||||
if values["OUTPUT_COMMENTS"]:
|
||||
comment = create_comment(values, "Begin toolchange")
|
||||
gcode += f"{linenumber(values)}{comment}{nl}"
|
||||
gcode.append(f"{linenumber(values)}{comment}{nl}")
|
||||
if values["OUTPUT_TOOL_CHANGE"]:
|
||||
if values["STOP_SPINDLE_FOR_TOOL_CHANGE"]:
|
||||
# stop the spindle
|
||||
gcode += f"{linenumber(values)}M5{nl}"
|
||||
gcode.append(f"{linenumber(values)}M5{nl}")
|
||||
for line in values["TOOL_CHANGE"].splitlines(False):
|
||||
gcode += f"{linenumber(values)}{line}{nl}"
|
||||
gcode.append(f"{linenumber(values)}{line}{nl}")
|
||||
elif values["OUTPUT_COMMENTS"]:
|
||||
# convert the tool change to a comment
|
||||
comment = create_comment(
|
||||
@@ -192,7 +192,7 @@ def check_for_tool_change(
|
||||
+ format_command_line(values, command_line)
|
||||
+ values["COMMAND_SPACE"],
|
||||
)
|
||||
gcode += f"{linenumber(values)}{comment}{nl}"
|
||||
gcode.append(f"{linenumber(values)}{comment}{nl}")
|
||||
return True
|
||||
return False
|
||||
|
||||
@@ -676,7 +676,7 @@ def parse_a_group(values: Values, gcode: Gcode, pathobj) -> None:
|
||||
if hasattr(pathobj, "Group"): # We have a compound or project.
|
||||
if values["OUTPUT_COMMENTS"]:
|
||||
comment = create_comment(values, f"Compound: {pathobj.Label}")
|
||||
gcode += f"{linenumber(values)}{comment}{nl}"
|
||||
gcode.append(f"{linenumber(values)}{comment}{nl}")
|
||||
for p in pathobj.Group:
|
||||
parse_a_group(values, gcode, p)
|
||||
else: # parsing simple path
|
||||
@@ -685,7 +685,7 @@ def parse_a_group(values: Values, gcode: Gcode, pathobj) -> None:
|
||||
return
|
||||
if values["OUTPUT_PATH_LABELS"] and values["OUTPUT_COMMENTS"]:
|
||||
comment = create_comment(values, f"Path: {pathobj.Label}")
|
||||
gcode += f"{linenumber(values)}{comment}{nl}"
|
||||
gcode.append(f"{linenumber(values)}{comment}{nl}")
|
||||
parse_a_path(values, gcode, pathobj)
|
||||
|
||||
|
||||
@@ -799,12 +799,12 @@ def parse_a_path(values: Values, gcode: Gcode, pathobj) -> None:
|
||||
command_line[0],
|
||||
] # swap the order of the commands
|
||||
# Add a line number to the front and a newline to the end of the command line
|
||||
gcode += (
|
||||
gcode.append(
|
||||
f"{linenumber(values)}{format_command_line(values, swapped_command_line)}{nl}"
|
||||
)
|
||||
else:
|
||||
# Add a line number to the front and a newline to the end of the command line
|
||||
gcode += f"{linenumber(values)}{format_command_line(values, command_line)}{nl}"
|
||||
gcode.append(f"{linenumber(values)}{format_command_line(values, command_line)}{nl}")
|
||||
|
||||
check_for_tlo(values, gcode, command, c.Parameters)
|
||||
check_for_machine_specific_commands(values, gcode, command)
|
||||
|
||||
Reference in New Issue
Block a user