Added an RMB option to Scale->Normalize in AutoUV window #625
Open
Micheus wants to merge 2 commits intodgud:masterfrom
Open
Added an RMB option to Scale->Normalize in AutoUV window #625Micheus wants to merge 2 commits intodgud:masterfrom
Micheus wants to merge 2 commits intodgud:masterfrom
Conversation
Collaborator
Author
|
Here is how it is working: |
Micheus
changed the title
Micheus
force-pushed
the
mv/new-uv-scale-normalize-RMB
branch
from
d7d25fd to
7d5b3c4
Compare
Owner
|
I have let co-pilot review this, fix stuff that seems valid and write when your are done, or nothing needs fixing. |
There was a problem hiding this comment.
Pull request overview
This PR adds a new Right Mouse Button (RMB) option to the AutoUV window's Scale -> Normalize menu that allows normalizing a set of charts relative to a reference chart. This is useful when remapping portions of already unwrapped objects or unwrapping composite objects in parts, ensuring new charts remain consistent in scale with previously unwrapped ones.
Key Changes
- Refactored the normalize menu structure to support both standard normalization (LMB) and reference-based normalization (RMB)
- Added new
normalize_by_ref/2function that calculates scaling based on a user-selected reference chart - Implemented interactive chart selection workflow with validation to ensure only unselected charts can be used as references
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
It's intended to normalize a set of charts relative to a reference one. This is option is useful in cases where we need to remap a portion of an already unwrapped object, or when someone unwraps a composite object in parts. In these situations, we want the new charts to remain consistent in scale with the previously unwrapped ones. To achieve this, we simply select one of them as a reference. NOTE: New AutoUV option to Normalize a set of charts using another as reference. It's accessed via RMB option. Thanks to envelupo and Vaughan for the suggestion
Micheus
force-pushed
the
mv/new-uv-scale-normalize-RMB
branch
from
7d5b3c4 to
829a868
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
It's intended to normalize a set of charts relative to a reference one.
This is option is useful in cases where we need to remap a portion of an already unwrapped object, or when someone unwraps a composite object in parts. In these situations, we want the new charts to remain consistent in scale with the previously unwrapped ones. To achieve this, we simply select one of them as a reference.
NOTE:
New AutoUV option to Normalize a set of charts using another as reference. It's accessed via RMB option. Thanks to envelupo and Vaughan for the suggestion