flashforge-python-api 1.3.1__py3-none-any.whl → 1.3.3__py3-none-any.whl

flashforge/client.py

@@ -6,9 +6,11 @@ communication layers for controlling FlashForge 3D printers.
 """
 
 import asyncio
+import logging
 from dataclasses import dataclass
 
 import aiohttp
+from pydantic import ValidationError
 
 from .api.constants.endpoints import CAMERA_STREAM_PORT, Endpoints
 from .api.controls import Control, Files, Info, JobControl, TempControl
@@ -19,6 +21,8 @@ from .tcp import FlashForgeClient as TcpClient
 from .tcp import FlashForgeTcpClientOptions, PrinterInfo
 from .tcp.parsers.temp_info import TempInfo
 
+logger = logging.getLogger(__name__)
+
 
 @dataclass(slots=True)
 class FiveMClientConnectionOptions:
@@ -160,7 +164,7 @@ class FlashForgeClient:
             True if TCP operations are permitted, False otherwise
         """
         if self._http_only:
-            print(f"{op}() unavailable: printer has no TCP control channel (HTTP-only).")
+            logger.debug("%s() unavailable: printer has no TCP control channel (HTTP-only).", op)
             return False
         return True
 
@@ -208,7 +212,7 @@ class FlashForgeClient:
         connected = await self.verify_connection()
         if connected:
             return True
-        print("Failed to connect to printer")
+        logger.warning("Failed to connect to the printer.")
         return False
 
     @property
@@ -256,7 +260,7 @@ class FlashForgeClient:
         """
         if await self.send_product_command():
             return await self.tcp_client.init_control()
-        print("New API control failed!")
+        logger.warning("New API control failed; the product command was rejected.")
         return False
 
     async def dispose(self) -> None:
@@ -392,13 +396,13 @@ class FlashForgeClient:
             # Get HTTP API response
             response = await self.info.get_detail_response()
             if not response or not NetworkUtils.is_ok(response):
-                print("Failed to get valid response from printer API")
+                logger.warning("Failed to get a valid response from the printer API.")
                 return False
 
             # Parse machine info from detail response
             machine_info = MachineInfoParser.from_detail(response.detail)
             if not machine_info:
-                print("Failed to parse machine info from detail response")
+                logger.warning("Failed to parse machine info from the /detail response.")
                 return False
 
             # Detect http_only from the parsed model BEFORE touching TCP. The
@@ -422,16 +426,16 @@ class FlashForgeClient:
                     ):
                         self.is_pro = True
                 else:
-                    print(
-                        "Warning: Unable to get PrinterInfo from TCP API, "
-                        "some features might not work"
+                    logger.warning(
+                        "Unable to get PrinterInfo from the TCP API; some features might "
+                        "not work."
                     )
 
             # Cache the details
             return self.cache_details(machine_info)
 
         except Exception as error:
-            print(f"Error in verify_connection: {error}")
+            logger.warning("Error in verify_connection: %s", error)
             return False
 
     async def send_product_command(self) -> bool:
@@ -476,8 +480,21 @@ class FlashForgeClient:
                     self._apply_feature_overrides()
                     return True
 
+        except ValidationError as error:
+            # Distinct from a rejected check code, and it must not read as one:
+            # the printer answered 200 with code 0, we simply could not parse
+            # what it said. Callers treat a False return as bad credentials
+            # (ff-5mp-hass surfaces it as "check code incorrect"), so the log
+            # line is the only place the real cause can surface.
+            logger.warning(
+                "Could not parse the /product response; reporting the printer as "
+                "unavailable rather than the credentials as rejected. This usually "
+                "means the firmware added a field this library has not seen. %s",
+                error,
+            )
+            return False
         except Exception as error:
-            print(f"Error in send_product_command: {error}")
+            logger.warning("Error in send_product_command: %s", error)
             return False
         finally:
             self._http_client_busy = False

flashforge/discovery/discovery.py

@@ -303,7 +303,9 @@ class PrinterDiscovery:
             try:
                 transport, _ = await self._create_endpoint(message_queue, error_queue)
                 self._send_discovery_packets(transport, config)
-                discovered_printers = await self._receive_responses(message_queue, error_queue, config)
+                discovered_printers = await self._receive_responses(
+                    message_queue, error_queue, config
+                )
 
                 for printer in discovered_printers:
                     key = f"{printer.ip_address}:{printer.command_port}"
@@ -454,7 +456,9 @@ class PrinterDiscovery:
         status_code = int.from_bytes(buffer[0x8A:0x8C], byteorder="big")
         product_type = int.from_bytes(buffer[0x8C:0x8E], byteorder="big")
         event_port = int.from_bytes(buffer[0x8E:0x90], byteorder="big")
-        serial_number = buffer[0x92 : 0x92 + 128].decode("utf-8", errors="ignore").split("\x00", 1)[0]
+        serial_number = (
+            buffer[0x92 : 0x92 + 128].decode("utf-8", errors="ignore").split("\x00", 1)[0]
+        )
         model = self.detect_modern_model(name, product_type, product_id)
 
         return DiscoveredPrinter(
@@ -586,9 +590,7 @@ class PrinterDiscovery:
                         continue
 
                     mask_int = (0xFFFFFFFF << (32 - prefix)) & 0xFFFFFFFF
-                    netmask = ".".join(
-                        str((mask_int >> (8 * (3 - i))) & 0xFF) for i in range(4)
-                    )
+                    netmask = ".".join(str((mask_int >> (8 * (3 - i))) & 0xFF) for i in range(4))
 
                     broadcast = self.calculate_broadcast_address(ip_addr, netmask)
                     if broadcast and broadcast not in broadcast_addresses:
@@ -695,7 +697,9 @@ class FlashForgePrinterDiscovery:
             return None
 
         if LEGACY_PROTOCOL_SIZE <= len(response) < MODERN_PROTOCOL_SIZE and len(response) >= 0x92:
-            legacy_name = response[0:32].decode("utf-8", errors="ignore").split("\x00", 1)[0].strip()
+            legacy_name = (
+                response[0:32].decode("utf-8", errors="ignore").split("\x00", 1)[0].strip()
+            )
             legacy_serial = (
                 response[0x92 : 0x92 + 32]
                 .decode("utf-8", errors="ignore")

flashforge/models/machine_info.py

@@ -29,6 +29,10 @@ class MachineState(Enum):
 class Temperature(BaseModel):
     """Represents a pair of current and target temperatures for a component like an extruder or print bed."""
 
+    # `extra="forbid"` deliberately: this model is never parsed from a printer
+    # payload, only constructed by MachineInfoParser.from_detail. Forbidding
+    # extras there catches a typo'd keyword argument in our own parser, which
+    # `extra="allow"` would silently absorb.
     model_config = ConfigDict(extra="forbid")
 
     current: float = Field(
@@ -42,7 +46,12 @@ class Temperature(BaseModel):
 class SlotInfo(BaseModel):
     """Information about a single slot in the material station."""
 
-    model_config = ConfigDict(extra="forbid", populate_by_name=True)
+    # `extra="allow"`: inbound, nested under FFPrinterDetail.matl_station_info.
+    # FFPrinterDetail allowing extras is not enough on its own - a new field on
+    # a *child* fails validation for the whole /detail response, and
+    # `Info.get_detail_response` swallows that and returns None, which the HA
+    # integration reports as "could not retrieve printer information".
+    model_config = ConfigDict(extra="allow", populate_by_name=True)
 
     has_filament: bool = Field(
         alias="hasFilament", description="Indicates if filament is present in this slot"
@@ -69,7 +78,8 @@ class SlotInfo(BaseModel):
 class MatlStationInfo(BaseModel):
     """Detailed information about the material station."""
 
-    model_config = ConfigDict(extra="forbid", populate_by_name=True)
+    # `extra="allow"`: inbound, see SlotInfo.
+    model_config = ConfigDict(extra="allow", populate_by_name=True)
 
     current_load_slot: int = Field(
         alias="currentLoadSlot", ge=0, le=4, description="Currently loading slot ID (0 if none)"
@@ -98,7 +108,8 @@ class MatlStationInfo(BaseModel):
 class IndepMatlInfo(BaseModel):
     """Information related to independent material loading, often used when a single extruder printer has a material station."""
 
-    model_config = ConfigDict(extra="forbid", populate_by_name=True)
+    # `extra="allow"`: inbound, see SlotInfo.
+    model_config = ConfigDict(extra="allow", populate_by_name=True)
 
     material_color: str = Field(alias="materialColor", description="Color of the material")
     material_name: str = Field(
@@ -123,7 +134,10 @@ class IndepMatlInfo(BaseModel):
 class FFGcodeToolData(BaseModel):
     """Represents data for a single tool/material used in a G-code file, typically part of a multi-material print."""
 
-    model_config = ConfigDict(extra="forbid", populate_by_name=True)
+    # `extra="allow"`: inbound, nested under FFGcodeFileEntry - which already
+    # allows extras, but a child that forbids them reintroduces the same
+    # names-only fallback it was flipped to avoid.
+    model_config = ConfigDict(extra="allow", populate_by_name=True)
 
     filament_weight: float = Field(
         alias="filamentWeight",
@@ -158,7 +172,11 @@ class FFGcodeToolData(BaseModel):
 class FFGcodeFileEntry(BaseModel):
     """Represents a single G-code file entry as returned by the /gcodeList endpoint, especially for printers like AD5X that provide detailed material info."""
 
-    model_config = ConfigDict(extra="forbid", populate_by_name=True)
+    # `extra="allow"`, matching FFPrinterDetail: a firmware update that adds one
+    # field must not fail validation, because Files.get_recent_file_list falls
+    # back to a names-only list when it does - silently dropping print time,
+    # filament weight, and the per-tool material data for EVERY file.
+    model_config = ConfigDict(extra="allow", populate_by_name=True)
 
     gcode_file_name: str = Field(
         alias="gcodeFileName", description="The name of the G-code file (e.g., 'FISH_PLA.3mf')"
@@ -231,6 +249,10 @@ class FFPrinterDetail(BaseModel):
     fill_amount: float | None = Field(default=None, ge=0, le=100, alias="fillAmount")
     firmware_version: str | None = Field(default=None, alias="firmwareVersion")
     flash_register_code: str | None = Field(default=None, alias="flashRegisterCode")
+    # AD5X-only, and firmware omits what does not apply: the Creator 5 series
+    # leaves this out of /detail even with four loaded slots, so None means "not
+    # reported", NOT "no station". Never gate a feature on it - read the derived
+    # FFMachineInfo.has_matl_station, or check matl_station_info yourself.
     has_matl_station: bool | None = Field(default=None, alias="hasMatlStation")
     matl_station_info: MatlStationInfo | None = Field(default=None, alias="matlStationInfo")
     indep_matl_info: IndepMatlInfo | None = Field(default=None, alias="indepMatlInfo")
@@ -289,6 +311,10 @@ class FFMachineInfo(BaseModel):
     This interface is populated by transforming data from FFPrinterDetail.
     """
 
+    # `extra="forbid"` deliberately: see Temperature. Built only at
+    # `MachineInfoParser.from_detail`, from ~50 keyword arguments, never from
+    # raw printer JSON - so forbidding extras is a typo check on our own code,
+    # not a firmware-compatibility risk.
     model_config = ConfigDict(extra="forbid")
 
     # Auto-shutdown settings
@@ -395,8 +421,15 @@ class FFMachineInfo(BaseModel):
     formatted_run_time: str = "00:00"
     formatted_total_run_time: str = "0h:0m"
 
-    # AD5X Material Station
-    has_matl_station: bool | None = None
+    # Material Station (AD5X / Creator 5 series)
+    #
+    # A capability, not a passthrough: MachineInfoParser DERIVES this from the
+    # station data rather than copying the raw `hasMatlStation` field, which the
+    # Creator 5 series never reports even with a station attached. It is a plain
+    # bool on purpose - there is no "unknown" state to represent, and offering
+    # one is what let an unreported flag read as absent hardware. For the
+    # untouched firmware value, read FFPrinterDetail.has_matl_station.
+    has_matl_station: bool = False
     matl_station_info: MatlStationInfo | None = None
     indep_matl_info: IndepMatlInfo | None = None
 

flashforge/models/responses.py

@@ -10,7 +10,12 @@ from .machine_info import FFGcodeFileEntry, FFPrinterDetail
 class GenericResponse(BaseModel):
     """Represents a generic response from the printer's API."""
 
-    model_config = ConfigDict(extra="forbid")
+    # `extra="allow"` on every inbound model: this is the envelope each printer
+    # response inherits, so a firmware update that adds one top-level key must
+    # not fail validation here. Callers cannot tell a parse failure from a real
+    # error - `send_product_command` returns a bare False either way, which the
+    # HA integration reports to the user as "check code incorrect".
+    model_config = ConfigDict(extra="allow")
 
     code: int
     message: str = ""
@@ -19,8 +24,6 @@ class GenericResponse(BaseModel):
 class DetailResponse(GenericResponse):
     """Represents the structure of the response from the printer's detail endpoint."""
 
-    model_config = ConfigDict(extra="forbid")
-
     detail: FFPrinterDetail
 
 
@@ -36,7 +39,10 @@ class Product(BaseModel):
     Field names match the actual camelCase format returned by the printer.
     """
 
-    model_config = ConfigDict(extra="forbid")
+    # `extra="allow"`: an unrecognized control-state flag must not fail
+    # validation. `send_product_command` has no fallback - it returns False,
+    # which is indistinguishable from the printer rejecting the credentials.
+    model_config = ConfigDict(extra="allow")
 
     chamberTempCtrlState: int  # noqa: N815 - field must match API camelCase response format
     externalFanCtrlState: int  # noqa: N815 - field must match API camelCase response format
@@ -55,8 +61,6 @@ class ProductResponse(GenericResponse):
     and a nested `product` object containing specific control states.
     """
 
-    model_config = ConfigDict(extra="forbid")
-
     product: Product
 
 
@@ -96,6 +100,10 @@ class FilamentArgs(BaseModel):
 class AD5XMaterialMapping(BaseModel):
     """Represents a material mapping for AD5X multi-color printing. Maps a tool (extruder) to a specific material station slot."""
 
+    # `extra="forbid"` deliberately, as on every outbound param model below:
+    # these are request bodies this library constructs, so no firmware update
+    # can break them, and forbidding extras turns a caller's typo'd keyword
+    # into an error instead of a field the printer silently ignores.
     model_config = ConfigDict(extra="forbid", populate_by_name=True)
 
     tool_id: int = Field(ge=0, le=3, description="Tool ID (0-based: 0, 1, 2, 3)")
@@ -181,9 +189,7 @@ class Creator5JobParams(BaseModel):
     leveling_before_print: bool = Field(
         description="Whether to perform bed leveling before printing"
     )
-    flow_calibration: bool = Field(
-        default=False, description="Whether to enable flow calibration"
-    )
+    flow_calibration: bool = Field(default=False, description="Whether to enable flow calibration")
     time_lapse_video: bool = Field(
         default=False, description="Whether to enable time lapse video recording"
     )
@@ -209,15 +215,11 @@ class Creator5UploadParams(BaseModel):
     leveling_before_print: bool = Field(
         description="Whether to perform bed leveling before printing"
     )
-    flow_calibration: bool = Field(
-        default=False, description="Whether to enable flow calibration"
-    )
+    flow_calibration: bool = Field(default=False, description="Whether to enable flow calibration")
     time_lapse_video: bool = Field(
         default=False, description="Whether to enable time lapse video recording"
     )
-    use_matl_station: bool = Field(
-        description="Whether this is a multi-tool material-station job"
-    )
+    use_matl_station: bool = Field(description="Whether this is a multi-tool material-station job")
     gcode_tool_cnt: int = Field(
         ge=1, le=4, description="Number of tools in the G-code (1-4 for the C5)"
     )
@@ -226,7 +228,9 @@ class Creator5UploadParams(BaseModel):
 class GCodeListResponse(GenericResponse):
     """Represents the response structure for a G-code file list request."""
 
-    model_config = ConfigDict(extra="forbid", populate_by_name=True)
+    # `extra="allow"`: see FFGcodeFileEntry. An unknown field at either level
+    # costs the caller all per-file metadata, so neither level may forbid one.
+    model_config = ConfigDict(extra="allow", populate_by_name=True)
 
     gcode_list: list[str] | list[FFGcodeFileEntry] | None = Field(default=None, alias="gcodeList")
     gcode_list_detail: list[FFGcodeFileEntry] | None = Field(default=None, alias="gcodeListDetail")
@@ -235,7 +239,8 @@ class GCodeListResponse(GenericResponse):
 class ThumbnailResponse(GenericResponse):
     """Represents the response structure for a G-code thumbnail request."""
 
-    model_config = ConfigDict(extra="forbid", populate_by_name=True)
+    # `extra="allow"`: inbound, see GenericResponse.
+    model_config = ConfigDict(extra="allow", populate_by_name=True)
 
     image_data: str = Field(
         alias="imageData", description="The thumbnail image data encoded as a base64 string"

flashforge/tcp/a3_client.py

@@ -84,7 +84,11 @@ class FlashForgeA3Client(FlashForgeTcpClient):
         bare_cmd = self._strip_protocol_prefix(cmd)
         if bare_cmd.startswith("M661"):
             return 500
-        if bare_cmd.startswith("M115") or bare_cmd.startswith("M119") or bare_cmd.startswith("M650"):
+        if (
+            bare_cmd.startswith("M115")
+            or bare_cmd.startswith("M119")
+            or bare_cmd.startswith("M650")
+        ):
             return 250
         return 200
 
@@ -102,7 +106,7 @@ class FlashForgeA3Client(FlashForgeTcpClient):
         if "Error: File not exists" in text_prefix:
             return True
 
-        magic_offset = buffer.find(b"\xA2\xA2\x2A\x2A")
+        magic_offset = buffer.find(b"\xa2\xa2\x2a\x2a")
         if magic_offset == -1 or len(buffer) < magic_offset + 8:
             return False
 
@@ -163,7 +167,9 @@ class FlashForgeA3Client(FlashForgeTcpClient):
             elif line.startswith("Mac Address:"):
                 info.mac_address = line.replace("Mac Address:", "", 1).strip()
             else:
-                volume_match = re.search(r"X:\s*(\d+)\s+Y:\s*(\d+)\s+Z:\s*(\d+)", line, re.IGNORECASE)
+                volume_match = re.search(
+                    r"X:\s*(\d+)\s+Y:\s*(\d+)\s+Z:\s*(\d+)", line, re.IGNORECASE
+                )
                 if volume_match:
                     info.build_volume = A3BuildVolume(
                         x=int(volume_match.group(1)),
@@ -331,7 +337,11 @@ class FlashForgeA3Client(FlashForgeTcpClient):
 
         files: list[A3FileEntry] = []
         count_index = next(
-            (index for index, line in enumerate(lines) if re.search(r"info_list\.size:\s*\d+", line)),
+            (
+                index
+                for index, line in enumerate(lines)
+                if re.search(r"info_list\.size:\s*\d+", line)
+            ),
             -1,
         )
         if count_index == -1:
@@ -354,11 +364,13 @@ class FlashForgeA3Client(FlashForgeTcpClient):
         if "Error: File not exists" in error_text:
             return None
 
-        magic_offset = buffer.find(b"\xA2\xA2\x2A\x2A")
+        magic_offset = buffer.find(b"\xa2\xa2\x2a\x2a")
         if magic_offset == -1 or len(buffer) < magic_offset + 8:
             return None
 
-        payload_length = int.from_bytes(buffer[magic_offset + 4 : magic_offset + 8], byteorder="big")
+        payload_length = int.from_bytes(
+            buffer[magic_offset + 4 : magic_offset + 8], byteorder="big"
+        )
         payload_end = magic_offset + 8 + payload_length
         if len(buffer) < payload_end:
             return None
@@ -378,7 +390,11 @@ class FlashForgeA3Client(FlashForgeTcpClient):
         return normalized.rstrip()
 
     def _get_normalized_lines(self, response: str) -> list[str]:
-        return [line.strip() for line in self._normalize_a3_text_response(response).split("\n") if line.strip()]
+        return [
+            line.strip()
+            for line in self._normalize_a3_text_response(response).split("\n")
+            if line.strip()
+        ]
 
     def _map_controller_command(self, cmd: str) -> str:
         if cmd == "~M146 r255 g255 b255 F0":
@@ -399,7 +415,10 @@ class FlashForgeA3Client(FlashForgeTcpClient):
         if bare_cmd.startswith("M23"):
             return "File opened" in normalized or "ok" in normalized
         if bare_cmd.startswith("M112"):
-            return bool(re.search(r"Emergency Stop", normalized, re.IGNORECASE)) or "Received." in normalized
+            return (
+                bool(re.search(r"Emergency Stop", normalized, re.IGNORECASE))
+                or "Received." in normalized
+            )
         if "ok" in normalized or "Received." in normalized:
             return True
         return normalized.startswith(bare_cmd)

flashforge/tcp/a4_client.py

@@ -274,7 +274,11 @@ class FlashForgeA4Client(FlashForgeTcpClient):
         return normalized
 
     def _get_normalized_lines(self, response: str) -> list[str]:
-        return [line.strip() for line in self._normalize_a4_text_response(response).split("\n") if line.strip()]
+        return [
+            line.strip()
+            for line in self._normalize_a4_text_response(response).split("\n")
+            if line.strip()
+        ]
 
     def _detect_variant(self, machine_type: str) -> A4PrinterVariant:
         normalized_type = machine_type.upper()

flashforge/tcp/parsers/endstop_status.py

@@ -4,10 +4,13 @@ FlashForge Python API - Endstop Status Parser
 Parses endstop and machine status information from M119 command responses.
 """
 
+import logging
 import re
 from enum import Enum
 from typing import Optional
 
+logger = logging.getLogger(__name__)
+
 # Pre-compiled regex to match key:value pairs where value is an integer
 KV_PATTERN = re.compile(r"([A-Za-z0-9-]+):\s*(\d+)")
 
@@ -118,7 +121,9 @@ class EndstopStatus:
             return None
 
         try:
-            lines = [line.strip() for line in replay.replace("\r", "\n").split("\n") if line.strip()]
+            lines = [
+                line.strip() for line in replay.replace("\r", "\n").split("\n") if line.strip()
+            ]
             if not lines:
                 return None
 
@@ -142,7 +147,7 @@ class EndstopStatus:
                     elif "BUSY" in machine_status:
                         self.machine_status = MachineStatus.BUSY
                     else:
-                        print(f"EndstopStatus: Encountered unknown MachineStatus: {machine_status}")
+                        logger.warning("EndstopStatus: unknown MachineStatus: %s", machine_status)
                         self.machine_status = MachineStatus.DEFAULT
                 elif line.startswith("MoveMode:"):
                     move_mode = line.replace("MoveMode:", "", 1).strip().upper()
@@ -157,7 +162,7 @@ class EndstopStatus:
                     elif "HOMING" in move_mode:
                         self.move_mode = MoveMode.HOMING
                     else:
-                        print(f"EndstopStatus: Encountered unknown MoveMode: {move_mode}")
+                        logger.warning("EndstopStatus: unknown MoveMode: %s", move_mode)
                         self.move_mode = MoveMode.DEFAULT
                 elif line.startswith("Status "):
                     self.status = Status(line)
@@ -182,9 +187,8 @@ class EndstopStatus:
             return self
 
         except Exception as e:
-            print("Unable to create EndstopStatus instance from replay")
-            print(f"Replay: {replay}")
-            print(f"Error: {e}")
+            logger.warning("Unable to create an EndstopStatus from the reply: %s", e)
+            logger.debug("Reply that failed to parse: %s", replay)
             return None
 
     def is_print_complete(self) -> bool:

flashforge/tcp/parsers/location_info.py

@@ -47,7 +47,9 @@ class LocationInfo:
             The populated LocationInfo instance, or None if parsing fails
         """
         try:
-            lines = [line.strip() for line in replay.replace("\r", "\n").split("\n") if line.strip()]
+            lines = [
+                line.strip() for line in replay.replace("\r", "\n").split("\n") if line.strip()
+            ]
             coordinate_line = next(
                 (line for line in lines if "X:" in line and "Y:" in line and "Z:" in line),
                 "",

flashforge/tcp/parsers/print_status.py

@@ -4,8 +4,11 @@ FlashForge Python API - Print Status Parser
 Parses print progress information from M27 command responses.
 """
 
+import logging
 from typing import Optional
 
+logger = logging.getLogger(__name__)
+
 
 class PrintStatus:
     """
@@ -48,7 +51,9 @@ class PrintStatus:
             self.sd_total = ""
             self.layer_current = ""
             self.layer_total = ""
-            lines = [line.strip() for line in replay.replace("\r", "\n").split("\n") if line.strip()]
+            lines = [
+                line.strip() for line in replay.replace("\r", "\n").split("\n") if line.strip()
+            ]
 
             for line in lines:
                 if "SD printing byte " in line:
@@ -67,13 +72,13 @@ class PrintStatus:
                         return None
 
             if not self.sd_current or not self.sd_total:
-                print("PrintStatus: Invalid SD progress format")
+                logger.warning("PrintStatus: invalid SD progress format.")
                 return None
 
             return self
 
         except Exception as e:
-            print(f"Error parsing print status: {e}")
+            logger.warning("Error parsing the print status: %s", e)
             return None
 
     def get_print_percent(self) -> float:

flashforge/tcp/parsers/printer_info.py

@@ -70,7 +70,9 @@ class PrinterInfo:
             return None
 
         try:
-            lines = [line.strip() for line in replay.replace("\r", "\n").split("\n") if line.strip()]
+            lines = [
+                line.strip() for line in replay.replace("\r", "\n").split("\n") if line.strip()
+            ]
 
             for line in lines:
                 if line.startswith("Machine Type:"):

flashforge/tcp/parsers/temp_info.py

@@ -127,7 +127,9 @@ class TempInfo:
             return None
 
         try:
-            lines = [line.strip() for line in replay.replace("\r", "\n").split("\n") if line.strip()]
+            lines = [
+                line.strip() for line in replay.replace("\r", "\n").split("\n") if line.strip()
+            ]
             if not lines:
                 logger.error("TempInfo replay has invalid data: %s", replay)
                 return None