Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Nodes] Add a new 'enabled' property to node's attributes #1007

Merged
merged 8 commits into from
Jul 29, 2020

Conversation

julien-hdev
Copy link
Contributor

@julien-hdev julien-hdev commented Jul 29, 2020

Description

'enabled' is a new parameter for the node's attributes. It allows to enable/disable the attribute. If it is disabled, the attribute will not be displayed inside the UI and will not be sent to the command line.

This is quite useful when an attribute is used only if another is checked, for instance.

'enabled' can receive either a boolean or a lambda function.

⚠️ Disabled parameters will not be used anymore in the UID. So it will change the invalidation status on existing projects. The major versions of the modified nodes are therefore changed.

The new property "enabled" allows to define a lambda to dynamically enable/disable parameters.
Disabled parameters are not used in the uid, not exported to the command line and not visible in the interface.
@fabiencastan
Copy link
Member

As disabled params will not be used anymore in the UID, it will change all the invalidation, so you need to increment the major version of each modified node.

Copy link
Member

@fabiencastan fabiencastan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to update the nodes versions

julien-hdev and others added 4 commits July 29, 2020 14:39
Params are only used on drag&drop and do not invalidate the node
computation.
Only Viewpoints and Intrinsics are now used in the invalidation.
@fabiencastan fabiencastan added the feature new feature (proposed as PR or issue planned by dev) label Jul 29, 2020
@fabiencastan fabiencastan added this to the Meshroom 2020.1.0 milestone Jul 29, 2020
@fabiencastan fabiencastan merged commit 6b99e0b into develop Jul 29, 2020
@fabiencastan fabiencastan deleted the dev/enabledAttr branch July 29, 2020 16:45
@fabiencastan fabiencastan changed the title [Nodes] Add a new 'enabled' parameter to node's attributes [Nodes] Add a new 'enabled' property to node's attributes Aug 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature new feature (proposed as PR or issue planned by dev)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants