-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[ui] Remove intrinsic if not used by any viewpoint #2395
[ui] Remove intrinsic if not used by any viewpoint #2395
Conversation
@@ -292,6 +292,24 @@ Panel { | |||
|
|||
function sendRemoveRequest() { | |||
if (!readOnly) { | |||
// look if the viewpoint's intrinsic is used by another viewpoint |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be done after the removeImageRequest, isn't it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe better to do it on the python side.
Instead of a generic call to reconstruction.removeAttribute(attribute), here:
https://github.com/alicevision/Meshroom/blob/7b691a05dd73c1aa1ee913aa11af84ba81d227b5/meshroom/ui/qml/WorkspaceView.qml#L81C17-L81C80
Add a removeImage and do all the checks in it (including the job done in allViewpointsCleared / removeAllImages).
https://github.com/alicevision/Meshroom/blob/7b691a05dd73c1aa1ee913aa11af84ba81d227b5/meshroom/ui/graph.py#L820C9-L820C24
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I created a removeImage to do checks of intrinsics, is it better ?
0e06232
to
c50d3cc
Compare
Description
When deleting image from the ImageGallery, if it is the last one using a specific intrinsic, it now removes the intrinsic.