diff --git a/plugins/CuraEngineBackend/StartSliceJob.py b/plugins/CuraEngineBackend/StartSliceJob.py index 4d924ac337..2887743b4f 100644 --- a/plugins/CuraEngineBackend/StartSliceJob.py +++ b/plugins/CuraEngineBackend/StartSliceJob.py @@ -1,4 +1,4 @@ -# Copyright (c) 2021-2022 Ultimaker B.V. +# Copyright (c) 2023 UltiMaker # Cura is released under the terms of the LGPLv3 or higher. import os @@ -24,6 +24,7 @@ from UM.Scene.Iterator.DepthFirstIterator import DepthFirstIterator from UM.Scene.Scene import Scene #For typing. from UM.Settings.Validator import ValidatorState from UM.Settings.SettingRelation import RelationType +from UM.Settings.SettingFunction import SettingFunction from cura.CuraApplication import CuraApplication from cura.Scene.CuraSceneNode import CuraSceneNode @@ -46,44 +47,60 @@ class StartJobResult(IntEnum): class GcodeStartEndFormatter(Formatter): - """Formatter class that handles token expansion in start/end gcode""" + # Formatter class that handles token expansion in start/end gcode + # Example of a start/end gcode string: + # ``` + # M104 S{material_print_temperature_layer_0, 0} ;pre-heat + # M140 S{material_bed_temperature_layer_0} ;heat bed + # M204 P{acceleration_print, 0} T{acceleration_travel, 0} + # M205 X{jerk_print, 0} + # ``` + # Any expression between curly braces will be evaluated and replaced with the result, using the + # context of the provided default extruder. If no default extruder is provided, the global stack + # will be used. Alternatively, if the expression is formatted as "{[expression], [extruder_nr]}", + # then the expression will be evaluated with the extruder stack of the specified extruder_nr. - def __init__(self, default_extruder_nr: int = -1) -> None: + _extruder_regex = re.compile(r"^\s*(?P.*)\s*,\s*(?P\d+)\s*$") + + def __init__(self, default_extruder_nr: int = -1, *, + additional_per_extruder_settings: Optional[Dict[str, Dict[str, any]]] = None) -> None: super().__init__() - self._default_extruder_nr = default_extruder_nr - - def get_value(self, key: str, args: str, kwargs: dict) -> str: #type: ignore # [CodeStyle: get_value is an overridden function from the Formatter class] - # The kwargs dictionary contains a dictionary for each stack (with a string of the extruder_nr as their key), - # and a default_extruder_nr to use when no extruder_nr is specified + self._default_extruder_nr: int = default_extruder_nr + self._additional_per_extruder_settings: Optional[Dict[str, Dict[str, any]]] = additional_per_extruder_settings + def get_value(self, expression: str, args: [str], kwargs: dict) -> str: extruder_nr = self._default_extruder_nr - key_fragments = [fragment.strip() for fragment in key.split(",")] - if len(key_fragments) == 2: - try: - extruder_nr = int(key_fragments[1]) - except ValueError: - try: - extruder_nr = int(kwargs["-1"][key_fragments[1]]) # get extruder_nr values from the global stack #TODO: How can you ever provide the '-1' kwarg? - except (KeyError, ValueError): - # either the key does not exist, or the value is not an int - Logger.log("w", "Unable to determine stack nr '%s' for key '%s' in start/end g-code, using global stack", key_fragments[1], key_fragments[0]) - elif len(key_fragments) != 1: - Logger.log("w", "Incorrectly formatted placeholder '%s' in start/end g-code", key) - return "{" + key + "}" + # The settings may specify a specific extruder to use. This is done by + # formatting the expression as "{expression}, {extruder_nr}". If the + # expression is formatted like this, we extract the extruder_nr and use + # it to get the value from the correct extruder stack. + match = self._extruder_regex.match(expression) + if match: + expression = match.group("expression") + extruder_nr = int(match.group("extruder_nr")) - key = key_fragments[0] + if self._additional_per_extruder_settings is not None and str( + extruder_nr) in self._additional_per_extruder_settings: + additional_variables = self._additional_per_extruder_settings[str(extruder_nr)] + else: + additional_variables = dict() - default_value_str = "{" + key + "}" - value = default_value_str - # "-1" is global stack, and if the setting value exists in the global stack, use it as the fallback value. - if key in kwargs["-1"]: - value = kwargs["-1"][key] - if str(extruder_nr) in kwargs and key in kwargs[str(extruder_nr)]: - value = kwargs[str(extruder_nr)][key] + # Add the arguments and keyword arguments to the additional settings. These + # are currently _not_ used, but they are added for consistency with the + # base Formatter class. + for key, value in enumerate(args): + additional_variables[key] = value + for key, value in kwargs.items(): + additional_variables[key] = value - if value == default_value_str: - Logger.log("w", "Unable to replace '%s' placeholder in start/end g-code", key) + if extruder_nr == -1: + container_stack = CuraApplication.getInstance().getGlobalContainerStack() + else: + container_stack = ExtruderManager.getInstance().getExtruderStack(extruder_nr) + + setting_function = SettingFunction(expression) + value = setting_function(container_stack, additional_variables=additional_variables) return value @@ -426,13 +443,14 @@ class StartSliceJob(Job): self._cacheAllExtruderSettings() try: - # any setting can be used as a token - fmt = GcodeStartEndFormatter(default_extruder_nr = default_extruder_nr) - if self._all_extruders_settings is None: - return "" - settings = self._all_extruders_settings.copy() - settings["default_extruder_nr"] = default_extruder_nr - return str(fmt.format(value, **settings)) + # Get "replacement-keys" for the extruders. In the formatter the settings stack is used to get the + # replacement values for the setting-keys. However, the values for `material_id`, `material_type`, + # etc are not in the settings stack. + additional_per_extruder_settings = self._all_extruders_settings.copy() + additional_per_extruder_settings["default_extruder_nr"] = default_extruder_nr + fmt = GcodeStartEndFormatter(default_extruder_nr=default_extruder_nr, + additional_per_extruder_settings=additional_per_extruder_settings) + return str(fmt.format(value)) except: Logger.logException("w", "Unable to do token replacement on start/end g-code") return str(value)