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

Viewer2D: Dynamically update the list of viewable outputs #2044

Merged
merged 3 commits into from
Jun 9, 2023

Conversation

cbentejac
Copy link
Contributor

Description

This PR splits from #1925 and keeps the dynamic update of the list of viewable outputs in the 2D Viewer: instead of displaying at all times the list of a node's viewable outputs independently from their current status (enabled or disabled), the 2D Viewer should only show the enabled outputs, and adding/removing them as soon as they get enabled/disabled.

Hiding (or showing) the connections of output attributes that get disabled (or enabled) will remain handled in #1925.

Features list

  • Update the list of viewable outputs in the 2D Viewer depending on the loaded node's enabled outputs.
  • Uncomment the conditional enabled status of some DepthMap's outputs.

@cbentejac cbentejac added this to the Meshroom 2023.2.0 milestone Jun 9, 2023
@cbentejac cbentejac requested review from fabiencastan and mugulmd June 9, 2023 07:16
@cbentejac cbentejac self-assigned this Jun 9, 2023
cbentejac and others added 3 commits June 9, 2023 14:22
…bled

When a node with viewable output attributes is loaded, the combo box  of
the 2D Viewer lists these attributes. As some of these attributes are
enabled or disabled, they need to be added or removed from that list,
without needing to load another node first, or to unload the node.

This commit connects the signal that is emitted when an output attribute
with a viewable output is enabled or disabled to the slot that reloads
the currently loaded node (and thus updates the list).
@mugulmd mugulmd force-pushed the dev/updateViewerList branch from 5d45cfa to 4e130e7 Compare June 9, 2023 13:12
@mugulmd mugulmd added UI invalidate:node Invalidates the cache of one or several nodes. labels Jun 9, 2023
@mugulmd mugulmd merged commit 840e15a into develop Jun 9, 2023
@mugulmd mugulmd deleted the dev/updateViewerList branch June 9, 2023 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalidate:node Invalidates the cache of one or several nodes. UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants