diff --git a/cura/BuildVolume.py b/cura/BuildVolume.py index a25604d165..91aa5a176d 100644 --- a/cura/BuildVolume.py +++ b/cura/BuildVolume.py @@ -338,13 +338,6 @@ class BuildVolume(SceneNode): prime_tower_x = self._global_container_stack.getProperty("prime_tower_position_x", "value") - machine_width / 2 prime_tower_y = - self._global_container_stack.getProperty("prime_tower_position_y", "value") + machine_depth / 2 - '''disallowed_areas.append([ - [prime_tower_x - prime_tower_size, prime_tower_y - prime_tower_size], - [prime_tower_x, prime_tower_y - prime_tower_size], - [prime_tower_x, prime_tower_y], - [prime_tower_x - prime_tower_size, prime_tower_y], - ])''' - self._prime_tower_area = Polygon([ [prime_tower_x - prime_tower_size, prime_tower_y - prime_tower_size], [prime_tower_x, prime_tower_y - prime_tower_size], diff --git a/cura/Settings/ContainerManager.py b/cura/Settings/ContainerManager.py index 5be134c74b..f7ea9bd416 100644 --- a/cura/Settings/ContainerManager.py +++ b/cura/Settings/ContainerManager.py @@ -172,11 +172,11 @@ class ContainerManager(QObject): containers = self._container_registry.findContainers(None, id=container_id) if not containers: UM.Logger.log("w", "Could not get metadata of container %s because it was not found.", container_id) - return False + return "" result = containers[0].getMetaDataEntry(entry_name) - if result: - return result + if result is not None: + return str(result) else: return "" diff --git a/cura/Settings/MachineManager.py b/cura/Settings/MachineManager.py index 0849786320..1324f3ab60 100644 --- a/cura/Settings/MachineManager.py +++ b/cura/Settings/MachineManager.py @@ -531,6 +531,12 @@ class MachineManager(QObject): containers[0].nameChanged.connect(self._onMaterialNameChanged) + if containers[0].getMetaDataEntry("compatible") == False: + message = Message(catalog.i18nc("@info:status", + "The selected material is imcompatible with the selected machine or configuration.")) + message.show() + + if old_quality: if old_quality_changes: new_quality = self._updateQualityChangesContainer(old_quality.getMetaDataEntry("quality_type"), old_quality_changes.getMetaDataEntry("name")) diff --git a/plugins/CuraEngineBackend/StartSliceJob.py b/plugins/CuraEngineBackend/StartSliceJob.py index fc26bd086b..efc36e6150 100644 --- a/plugins/CuraEngineBackend/StartSliceJob.py +++ b/plugins/CuraEngineBackend/StartSliceJob.py @@ -82,6 +82,13 @@ class StartSliceJob(Job): self.setResult(StartJobResult.SettingError) return + for extruder_stack in cura.Settings.ExtruderManager.getInstance().getMachineExtruders(stack.getId()): + material = extruder_stack.findContainer({"type": "material"}) + if material: + if material.getMetaDataEntry("compatible") == False: + self.setResult(StartJobResult.SettingError) + return + # Don't slice if there is a per object setting with an error value. for node in DepthFirstIterator(self._scene.getRoot()): if type(node) is not SceneNode or not node.isSelectable(): diff --git a/plugins/CuraProfileReader/CuraProfileReader.py b/plugins/CuraProfileReader/CuraProfileReader.py index 2bccb8e3cb..d1be5b59a8 100644 --- a/plugins/CuraProfileReader/CuraProfileReader.py +++ b/plugins/CuraProfileReader/CuraProfileReader.py @@ -54,10 +54,10 @@ class CuraProfileReader(ProfileReader): if not "general" in parser: Logger.log("w", "Missing required section 'general'.") - return None + return [] if not "version" in parser["general"]: Logger.log("w", "Missing required 'version' property") - return None + return [] version = int(parser["general"]["version"]) if InstanceContainer.Version != version: diff --git a/plugins/XmlMaterialProfile/XmlMaterialProfile.py b/plugins/XmlMaterialProfile/XmlMaterialProfile.py index be8f5146c9..1679e8f30d 100644 --- a/plugins/XmlMaterialProfile/XmlMaterialProfile.py +++ b/plugins/XmlMaterialProfile/XmlMaterialProfile.py @@ -83,7 +83,6 @@ class XmlMaterialProfile(UM.Settings.InstanceContainer): # machine / variant combination: only changes for itself. def serialize(self): if self._read_only: - Logger.log("w", "Serializing read-only container [%s], probably a programming error." % self.id) return registry = UM.Settings.ContainerRegistry.getInstance() @@ -428,8 +427,8 @@ class XmlMaterialProfile(UM.Settings.InstanceContainer): for identifier in identifiers: machine_id = self.__product_id_map.get(identifier.get("product"), None) if machine_id is None: - Logger.log("w", "Cannot create material for unknown machine %s", identifier.get("product")) - continue + # Lets try again with some naive heuristics. + machine_id = identifier.get("product").replace(" ", "").lower() definitions = UM.Settings.ContainerRegistry.getInstance().findDefinitionContainers(id = machine_id) if not definitions: @@ -454,6 +453,7 @@ class XmlMaterialProfile(UM.Settings.InstanceContainer): UM.Settings.ContainerRegistry.getInstance().addContainer(new_material) + hotends = machine.iterfind("./um:hotend", self.__namespaces) for hotend in hotends: hotend_id = hotend.get("id") @@ -482,14 +482,12 @@ class XmlMaterialProfile(UM.Settings.InstanceContainer): else: Logger.log("d", "Unsupported material setting %s", key) - if not hotend_compatibility: - continue - new_hotend_material = XmlMaterialProfile(self.id + "_" + machine_id + "_" + hotend_id.replace(" ", "_")) new_hotend_material.setName(self.getName()) new_hotend_material.setMetaData(copy.deepcopy(self.getMetaData())) new_hotend_material.setDefinition(definition) new_hotend_material.addMetaDataEntry("variant", variant_containers[0].id) + new_hotend_material.addMetaDataEntry("compatible", hotend_compatibility) for key, value in global_setting_values.items(): new_hotend_material.setProperty(key, "value", value, definition) @@ -507,7 +505,9 @@ class XmlMaterialProfile(UM.Settings.InstanceContainer): # Change the type of this container so it is not shown as an option in menus. # This uses InstanceContainer.setMetaDataEntry because otherwise all containers that # share this basefile are also updated. + dirty = self.isDirty() super().setMetaDataEntry("type", "incompatible_material") + super().setDirty(dirty) # reset dirty flag after setMetaDataEntry def _addSettingElement(self, builder, instance): try: diff --git a/resources/qml/SidebarHeader.qml b/resources/qml/SidebarHeader.qml index c9c0e722eb..6f9338d23c 100644 --- a/resources/qml/SidebarHeader.qml +++ b/resources/qml/SidebarHeader.qml @@ -232,6 +232,19 @@ Column text: Cura.MachineManager.activeMaterialName tooltip: Cura.MachineManager.activeMaterialName visible: Cura.MachineManager.hasMaterials + property var valueError: + { + var data = Cura.ContainerManager.getContainerMetaDataEntry(Cura.MachineManager.activeMaterialId, "compatible") + print(data) + if(data == "" || data == "True") + { + return false + } + if(data == "False") + { + return true + } + } enabled: !extrudersList.visible || base.currentExtruderIndex > -1 height: UM.Theme.getSize("setting_control").height diff --git a/resources/themes/cura/styles.qml b/resources/themes/cura/styles.qml index 91a512b6db..96bbf03a45 100644 --- a/resources/themes/cura/styles.qml +++ b/resources/themes/cura/styles.qml @@ -15,7 +15,11 @@ QtObject { { if(control.enabled) { - if(control.valueWarning) + if(control.valueError) + { + return Theme.getColor("setting_validation_error"); + } + else if(control.valueWarning) { return Theme.getColor("setting_validation_warning"); } else