| | import re |
| | from decimal import Decimal, getcontext |
| | import decimal |
| |
|
| | |
| | interpolation_commands = {"G01", "G02", "G03"} |
| | movement_commands = {"G00"} |
| |
|
| | |
| | gcode_pattern = re.compile( |
| | r"(G\d+|M\d+|X[-+]?\d*\.?\d+|Y[-+]?\d*\.?\d+|" |
| | r"Z[-+]?\d*\.?\d+|I[-+]?\d*\.?\d+|J[-+]?\d*\.?\d+|" |
| | r"F[-+]?\d*\.?\d+|S[-+]?\d*\.?\d+)" |
| | ) |
| |
|
| | def standardize_codes(line): |
| | """ |
| | Standardizes M-codes and G-codes to two digits by adding a leading zero if necessary. |
| | """ |
| | line = re.sub(r"\b(M|G)(\d)\b", r"\g<1>0\2", line) |
| | return line |
| |
|
| | def remove_comments(line): |
| | """ |
| | Removes comments from a G-code line. Supports both ';' and '()' style comments. |
| | """ |
| | |
| | line = line.split(';')[0] |
| | |
| | line = re.sub(r'\(.*?\)', '', line) |
| | return line.strip() |
| |
|
| | def preprocess_gcode(gcode): |
| | """ |
| | Removes comments from the G-code and returns a list of tuples (original_line_number, cleaned_line). |
| | Includes all lines to maintain accurate line numbering. |
| | """ |
| | cleaned_lines = [] |
| | lines = gcode.splitlines() |
| |
|
| | for idx, line in enumerate(lines): |
| | original_line_number = idx + 1 |
| | line = standardize_codes(line.strip()) |
| | |
| | line_no_comments = remove_comments(line) |
| | |
| | cleaned_lines.append((original_line_number, line_no_comments)) |
| |
|
| | return cleaned_lines |
| |
|
| | def check_required_gcodes(lines_with_numbers): |
| | """ |
| | Checks that the G-code contains required G-codes: G20/G21, G90/G91, G54-G59, and G17. |
| | Returns a list of errors with individual entries for each missing group. |
| | """ |
| | required_groups = { |
| | "units": {"G20", "G21"}, |
| | "mode": {"G90", "G91"}, |
| | "work_coordinates": {"G54", "G55", "G56", "G57", "G58", "G59"}, |
| | "plane": {"G17", "G18", "G19"}, |
| | } |
| |
|
| | |
| | found_codes = {} |
| | for original_line_number, line in lines_with_numbers: |
| | tokens = line.split() |
| | for token in tokens: |
| | found_codes.setdefault(token, original_line_number) |
| |
|
| | |
| | missing_group_errors = [] |
| | |
| | |
| | for category, codes in required_groups.items(): |
| | |
| | found = any(code in found_codes for code in codes) |
| | if not found: |
| | missing_codes = "/".join(sorted(codes)) |
| | |
| | for original_line_number, line in lines_with_numbers: |
| | if gcode_pattern.search(line): |
| | missing_group_errors.append((original_line_number, f"(Error) Missing required G-codes: ({category}) {missing_codes}")) |
| | break |
| | else: |
| | |
| | missing_group_errors.append((1, f"(Error) Missing required G-codes: ({category}) {missing_codes}")) |
| |
|
| | return missing_group_errors |
| |
|
| | def check_required_gcodes_position(lines_with_numbers): |
| | """ |
| | Ensures required G-codes appear before movement commands. |
| | Flags changes in critical settings (e.g., units) after movement commands. |
| | """ |
| | issues = [] |
| | movement_seen = False |
| | required_groups = { |
| | "units": {"G20", "G21"}, |
| | "mode": {"G90", "G91"}, |
| | "work_coordinates": {"G54", "G55", "G56", "G57", "G58", "G59"}, |
| | "plane": {"G17", "G18", "G19"}, |
| | } |
| | critical_gcodes = { |
| | "units": {"G20", "G21"}, |
| | "plane": {"G17", "G18", "G19"}, |
| | } |
| |
|
| | |
| | codes_before_movement = set() |
| |
|
| | for original_line_number, line in lines_with_numbers: |
| | tokens = line.split() |
| |
|
| | |
| | if not movement_seen and any(cmd in tokens for cmd in {"G00", "G01", "G02", "G03"}): |
| | movement_seen = True |
| |
|
| | if not movement_seen: |
| | |
| | codes_before_movement.update(tokens) |
| | else: |
| | |
| | for token in tokens: |
| | for category, codes in critical_gcodes.items(): |
| | if token in codes: |
| | issues.append((original_line_number, f"(Warning) {token} appears after movement commands. Ensure this change is intentional -> {line.strip()}")) |
| |
|
| | |
| | missing_groups = [] |
| | for category, codes in required_groups.items(): |
| | if not any(code in codes_before_movement for code in codes): |
| | missing_codes = "/".join(sorted(codes)) |
| | missing_groups.append(f"({category}) {missing_codes}") |
| |
|
| | if missing_groups: |
| | first_movement_line = next( |
| | (line_num for line_num, line in lines_with_numbers if any(cmd in line for cmd in {"G00", "G01", "G02", "G03"})), |
| | 1 |
| | ) |
| | issues.append((first_movement_line, f"(Error) Missing required G-codes before first movement: {', '.join(missing_groups)}")) |
| |
|
| | return issues |
| |
|
| | def check_end_gcode(lines_with_numbers): |
| | """ |
| | Checks that M30 is the last G-code command. |
| | Allows blank lines or '%' symbols after M30. |
| | """ |
| | found_m30 = False |
| |
|
| | |
| | errors = [] |
| |
|
| | for idx, (original_line_number, line) in enumerate(lines_with_numbers): |
| | if not line.strip() or line.strip() == "%": |
| | continue |
| |
|
| | if "M30" in line: |
| | if found_m30: |
| | errors.append((original_line_number, "(Error) M30 must be the last G-code command in the G-code.")) |
| | found_m30 = True |
| | continue |
| |
|
| | |
| | if found_m30 and gcode_pattern.search(line): |
| | errors.append((original_line_number, f"(Error) No G-code commands should appear after M30. Found '{line.strip()}'.")) |
| | |
| | if not found_m30: |
| | if lines_with_numbers: |
| | last_line_number = lines_with_numbers[-1][0] |
| | else: |
| | last_line_number = 1 |
| | errors.append((last_line_number, "(Error) M30 is missing from the G-code.")) |
| |
|
| | return errors |
| |
|
| | def check_spindle(lines_with_numbers): |
| | """ |
| | Checks spindle-related issues in the G-code. |
| | """ |
| | issues = [] |
| | spindle_on = False |
| | spindle_started = False |
| |
|
| | for idx, (original_line_number, line) in enumerate(lines_with_numbers): |
| | |
| | if not line.strip() or line.strip() == "%": |
| | continue |
| |
|
| | tokens = line.split() |
| |
|
| | |
| | if not gcode_pattern.search(line): |
| | issues.append((original_line_number, f"(Error) Invalid G-code command or syntax error -> {line.strip()}")) |
| |
|
| | |
| | if "M03" in tokens or "M04" in tokens: |
| | |
| | if spindle_on: |
| | issues.append((original_line_number, "(Warning) Spindle is already on.")) |
| |
|
| | |
| | s_value_present = any(token.startswith("S") for token in tokens) |
| | if not s_value_present: |
| | issues.append((original_line_number, "(Error) Spindle speed (S value) is missing when turning on the spindle with M03/M04.")) |
| |
|
| | spindle_on = True |
| | spindle_started = True |
| |
|
| | |
| | if "M05" in tokens: |
| | spindle_on = False |
| |
|
| | |
| | if any(cmd in tokens for cmd in interpolation_commands): |
| | if not spindle_on: |
| | issues.append((original_line_number, f"(Error) Move command without spindle on -> {line.strip()}")) |
| |
|
| | |
| | if spindle_on: |
| | last_line_number = lines_with_numbers[-1][0] |
| | issues.append((last_line_number, "(Error) Spindle was not turned off (M05) before the end of the program.")) |
| |
|
| | |
| | if not spindle_started: |
| | issues.append((0, "(Error) Spindle was never turned on in the G-code.")) |
| |
|
| | return issues |
| |
|
| | def check_feed_rate(lines_with_numbers): |
| | """ |
| | Checks feed rate related issues in the G-code. |
| | """ |
| | issues = [] |
| | last_feed_rate = None |
| | interpolation_command_seen = False |
| |
|
| | for idx, (original_line_number, line) in enumerate(lines_with_numbers): |
| | |
| | if not line.strip() or line.strip() == "%": |
| | continue |
| |
|
| | tokens = line.split() |
| | commands = set(tokens) |
| | feed_rates = [token for token in tokens if token.startswith("F")] |
| |
|
| | |
| | if feed_rates and not any(cmd in interpolation_commands for cmd in commands): |
| | issues.append((original_line_number, f"(Warning) Feed rate specified without interpolation command -> {line.strip()}")) |
| |
|
| | |
| | if any(cmd in commands for cmd in interpolation_commands): |
| | if not interpolation_command_seen: |
| | interpolation_command_seen = True |
| | if not feed_rates and last_feed_rate is None: |
| | issues.append((original_line_number, f"(Error) First interpolation command must have a feed rate -> {line.strip()}")) |
| | else: |
| | |
| | if feed_rates: |
| | last_feed_rate = feed_rates[-1] |
| | else: |
| | |
| | if feed_rates: |
| | current_feed_rate = feed_rates[-1] |
| | if current_feed_rate == last_feed_rate: |
| | issues.append((original_line_number, f"(Warning) Feed rate {current_feed_rate} is already set; no need to specify again.")) |
| | else: |
| | last_feed_rate = current_feed_rate |
| |
|
| | return issues |
| |
|
| | def check_depth_of_cut(lines_with_numbers, depth_max=0.1): |
| | """ |
| | Checks that all cutting moves on the Z-axis have a uniform depth and do not exceed the maximum depth. |
| | """ |
| | getcontext().prec = 6 |
| | depth_max = Decimal(str(depth_max)) |
| | issues = [] |
| |
|
| | positioning_mode = "G90" |
| | current_z = Decimal('0.0') |
| | depths = set() |
| | z_negative_seen = False |
| |
|
| | for idx, (original_line_number, line) in enumerate(lines_with_numbers): |
| | |
| | if not line.strip() or line.strip() == "%": |
| | continue |
| |
|
| | tokens = line.split() |
| |
|
| | if "G90" in tokens: |
| | positioning_mode = "G90" |
| | elif "G91" in tokens: |
| | positioning_mode = "G91" |
| |
|
| | if any(cmd in tokens for cmd in interpolation_commands.union(movement_commands)): |
| | z_values = [token for token in tokens if token.startswith("Z")] |
| | if z_values: |
| | try: |
| | z_value = Decimal(z_values[-1][1:]) |
| | except (ValueError, decimal.InvalidOperation): |
| | issues.append((original_line_number, f"(Error) Invalid Z value -> {line.strip()}")) |
| | continue |
| |
|
| | if positioning_mode == "G90": |
| | new_z = z_value |
| | elif positioning_mode == "G91": |
| | new_z = current_z + z_value |
| |
|
| | if new_z < Decimal('0.0'): |
| | z_negative_seen = True |
| | depth = abs(new_z) |
| | depth = depth.quantize(Decimal('0.0001')).normalize() |
| | depths.add(depth) |
| |
|
| | if depth > depth_max: |
| | issues.append((original_line_number, f"(Error) Depth of cut {depth} exceeds maximum allowed depth of {depth_max.normalize()} -> {line.strip()}")) |
| |
|
| | current_z = new_z |
| |
|
| | if z_negative_seen: |
| | if len(depths) > 1: |
| | depth_values = ', '.join(str(d.normalize()) for d in sorted(depths)) |
| | issues.append((0, f"(Warning) Inconsistent depths of cut detected: {depth_values}")) |
| | else: |
| | issues.append((0, "(Error) No cutting moves detected on the Z-axis.")) |
| |
|
| | return issues |
| |
|
| | def check_interpolation_depth(lines_with_numbers): |
| | """ |
| | Checks that all interpolation commands moving in X or Y are executed at a negative Z depth (i.e., cutting). |
| | Does not report errors for interpolation commands used for plunging or retracting (Z-axis movements only). |
| | """ |
| | getcontext().prec = 6 |
| | issues = [] |
| |
|
| | positioning_mode = "G90" |
| | current_z = Decimal('0.0') |
| |
|
| | for idx, (original_line_number, line) in enumerate(lines_with_numbers): |
| | |
| | if not line.strip() or line.strip() == "%": |
| | continue |
| |
|
| | tokens = line.split() |
| |
|
| | |
| | if "G90" in tokens: |
| | positioning_mode = "G90" |
| | elif "G91" in tokens: |
| | positioning_mode = "G91" |
| |
|
| | |
| | z_values = [token for token in tokens if token.startswith("Z")] |
| | if z_values: |
| | try: |
| | z_value = Decimal(z_values[-1][1:]) |
| | except (ValueError, decimal.InvalidOperation): |
| | issues.append((original_line_number, f"(Error) Invalid Z value -> {line.strip()}")) |
| | continue |
| |
|
| | |
| | if positioning_mode == "G90": |
| | current_z = z_value |
| | elif positioning_mode == "G91": |
| | current_z += z_value |
| |
|
| | |
| | if any(cmd in tokens for cmd in interpolation_commands): |
| | |
| | has_xy_movement = any(token.startswith(('X', 'Y')) for token in tokens) |
| | if has_xy_movement and current_z >= Decimal('0.0'): |
| | issues.append((original_line_number, f"(Warning) Interpolation command with XY movement executed without cutting depth (Z={current_z}) -> {line.strip()}")) |
| |
|
| | return issues |
| |
|
| | def check_plunge_retract_moves(lines_with_numbers): |
| | """ |
| | Checks that plunging and retracting moves along the Z-axis use G01 instead of G00. |
| | Reports an error if G00 is used for Z-axis movements to Z positions less than or equal to zero. |
| | """ |
| | issues = [] |
| | positioning_mode = "G90" |
| | current_z = None |
| |
|
| | for idx, (original_line_number, line) in enumerate(lines_with_numbers): |
| | |
| | if not line.strip() or line.strip() == "%": |
| | continue |
| |
|
| | tokens = line.split() |
| |
|
| | |
| | if "G90" in tokens: |
| | positioning_mode = "G90" |
| | elif "G91" in tokens: |
| | positioning_mode = "G91" |
| |
|
| | |
| | z_values = [token for token in tokens if token.startswith("Z")] |
| | if z_values: |
| | try: |
| | z_value = Decimal(z_values[-1][1:]) |
| | except (ValueError, decimal.InvalidOperation): |
| | issues.append((original_line_number, f"(Error) Invalid Z value -> {line.strip()}")) |
| | continue |
| |
|
| | |
| | if current_z is None: |
| | current_z = z_value |
| | else: |
| | if positioning_mode == "G90": |
| | current_z = z_value |
| | elif positioning_mode == "G91": |
| | current_z += z_value |
| |
|
| | |
| | |
| | if "G00" in tokens and current_z <= Decimal('0.0'): |
| | issues.append((original_line_number, f"(Error) G00 used for plunging to Z={current_z}. Use G01 to safely approach the workpiece -> {line.strip()}")) |
| |
|
| | return issues |
| |
|
| | def run_checks(gcode, depth_max=0.1): |
| | """ |
| | Runs all checks and returns a tuple containing lists of errors and warnings. |
| | """ |
| | errors = [] |
| | warnings = [] |
| |
|
| | |
| | lines_with_numbers = preprocess_gcode(gcode) |
| |
|
| | |
| | required_gcode_issues = check_required_gcodes(lines_with_numbers) |
| | required_gcode_position_issues = check_required_gcodes_position(lines_with_numbers) |
| | spindle_issues = check_spindle(lines_with_numbers) |
| | feed_rate_issues = check_feed_rate(lines_with_numbers) |
| | depth_issues = check_depth_of_cut(lines_with_numbers, depth_max) |
| | end_gcode_issues = check_end_gcode(lines_with_numbers) |
| | interpolation_depth_issues = check_interpolation_depth(lines_with_numbers) |
| | plunge_retract_issues = check_plunge_retract_moves(lines_with_numbers) |
| |
|
| | |
| | all_issues = ( |
| | required_gcode_issues |
| | + required_gcode_position_issues |
| | + spindle_issues |
| | + feed_rate_issues |
| | + depth_issues |
| | + end_gcode_issues |
| | + interpolation_depth_issues |
| | + plunge_retract_issues |
| | ) |
| |
|
| | |
| | for line_num, message in all_issues: |
| | if "(Error)" in message: |
| | errors.append((line_num, message)) |
| | elif "(Warning)" in message: |
| | warnings.append((line_num, message)) |
| |
|
| | |
| | errors.sort(key=lambda x: x[0]) |
| | warnings.sort(key=lambda x: x[0]) |
| |
|
| | return errors, warnings |
| |
|
| | if __name__ == "__main__": |
| | |
| | gcode_sample = """ |
| | % |
| | G21 G90 G17 G54 |
| | G00 X0 Y0 Z5.0 |
| | M03 S1000 |
| | G01 Z-0.1 F100 ; Plunge using rapid movement (should be G01) |
| | G54 |
| | G01 Z-0.1 |
| | G01 X10 Y10 |
| | G01 X20 Y20 |
| | G00 Z5.0 ; Retract using rapid movement (allowed since Z > 0) |
| | M05 |
| | M30 |
| | % |
| | """ |
| |
|
| | depth_max = 0.1 |
| | errors, warnings = run_checks(gcode_sample, depth_max) |
| |
|
| | |
| | output_lines = [] |
| | if errors or warnings: |
| | output_lines.append("Issues found in G-code:") |
| | for line_num, message in errors + warnings: |
| | if line_num > 0: |
| | output_lines.append(f"Line {line_num}: {message}") |
| | else: |
| | output_lines.append(message) |
| | print('\n'.join(output_lines)) |
| | else: |
| | print("Your G-code looks good!") |