[core] Attribute: Parent's exposed property takes precedence over description's

The `exposed` property, which determines whether the attribute is
displayed on the upper part of the node in the Graph Editor, is set
for each attribute individually in their node's description.

If an attribute has a parent (meaning it depends on a `GroupAttribute` or
a `ListAttribute`) whose `exposed` property value differs, it does not
make sense to display it separately from it. The attribute's `exposed`
should align with its parent's.

Author: cbentejac

meshroom/core/attribute.py

@@ -61,11 +61,14 @@ def __init__(self, node, attributeDesc, isOutput, root=None, parent=None):
         self._description = attributeDesc.description
         self._invalidate = False if self._isOutput else attributeDesc.invalidate
 
+        self._exposed = attributeDesc.exposed
         self._depth = 0
         if root is not None:
             current = self
             while current.root is not None:
                 self._depth += 1
+                if current.root.exposed != self._exposed:
+                    self._exposed = current.root.exposed
                 current = current.root
 
         # invalidation value for output attributes
@@ -87,6 +90,9 @@ def root(self):
     def getDepth(self):
         return self._depth
 
+    def getExposed(self):
+        return self._exposed
+
     def getName(self):
         """ Attribute name """
         return self._name
@@ -462,6 +468,7 @@ def getFlattenedChildren(self):
     type = Property(str, getType, constant=True)
     baseType = Property(str, getType, constant=True)
     isReadOnly = Property(bool, _isReadOnly, constant=True)
+    exposed = Property(bool, getExposed, constant=True)
 
     # Description of the attribute
     descriptionChanged = Signal()

meshroom/ui/qml/GraphEditor/Node.qml

@@ -434,7 +434,7 @@ Item {
 
                                 delegate: Loader {
                                     id: inputLoader
-                                    active: !object.isOutput && object.desc.exposed && object.desc.visible
+                                    active: !object.isOutput && object.exposed && object.desc.visible
                                     visible: Boolean(object.enabled)
                                     width: inputs.width
 
@@ -494,7 +494,7 @@ Item {
                                     model: node ? node.attributes : undefined
                                     delegate: Loader {
                                         id: paramLoader
-                                        active: !object.isOutput && !object.desc.exposed && object.desc.visible
+                                        active: !object.isOutput && !object.exposed && object.desc.visible
                                         visible: Boolean(object.enabled || object.isLink || object.hasOutputConnections)
                                         property bool isFullyActive: Boolean(m.displayParams || object.isLink || object.hasOutputConnections)
                                         width: parent.width