Copy/Paste UV Operator Sometimes doesn't Work #106314

New Issue

Closed

opened 2023-03-30 15:48:19 +02:00 by Omar · 4 comments

Omar commented

2023-03-30 15:48:19 +02:00

System Information
Operating system: Linux-5.19.0-35-generic-x86_64-with-glibc2.35 64 Bits
Graphics card: NVIDIA GeForce RTX 2060/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 515.86.01

Blender Version
Broken: version: 3.5.0, branch: blender-v3.5-release, commit date: 2023-03-29 02:56, hash: 1be25cfff18b
Worked: (newest version of Blender that worked as expected)

Short description of error
Copy/Paste UV is failing on a set of identical islands

Exact steps for others to reproduce the error
Select one island. In the UV Editor to UV->Copy UVs. Select the other island. In the UV Editor Select UV->Paste UVs. The island doesn't move to the first one.
Check Attached .blend file.

Added the alternative file from #106429 (unable to repro the issue with CopyUV.blend file)

**System Information** Operating system: Linux-5.19.0-35-generic-x86_64-with-glibc2.35 64 Bits Graphics card: NVIDIA GeForce RTX 2060/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 515.86.01 **Blender Version** Broken: version: 3.5.0, branch: blender-v3.5-release, commit date: 2023-03-29 02:56, hash: `1be25cfff18b` Worked: (newest version of Blender that worked as expected) **Short description of error** Copy/Paste UV is failing on a set of identical islands **Exact steps for others to reproduce the error** Select one island. In the UV Editor to UV->Copy UVs. Select the other island. In the UV Editor Select UV->Paste UVs. The island doesn't move to the first one. Check Attached .blend file. Added the alternative file from #106429 (unable to repro the issue with `CopyUV.blend` file)

CopyUV.blend

1.0 MiB

Test - Copy-Paste UV.blend

538 KiB

Omar added the

labels 2023-03-30 15:48:20 +02:00

Iliya Katushenock added the

label 2023-03-30 15:49:11 +02:00

Chris Blackbourn self-assigned this 2023-03-31 00:09:06 +02:00

Chris Blackbourn commented

2023-03-31 01:04:49 +02:00

Member

Looks like we're tripping this safe-guard to prevent an infinite loop:

    if (iteration_count++ > 10000000) {
      /* Unlikely to find a solution, may as well give up.
       * Can occur with moderate sized inputs where the graph has lots of symmetry, e.g. a cube
       * subdivided 3x times.
       */
      *inc_pos = 0;
      break;
    }

If you remove that, the "Paste" operation eventually completes successfully on iteration_count == 255762942, about 25x larger than the current threshold.

Looks like we're tripping this safe-guard to prevent an infinite loop: ``` if (iteration_count++ > 10000000) { /* Unlikely to find a solution, may as well give up. * Can occur with moderate sized inputs where the graph has lots of symmetry, e.g. a cube * subdivided 3x times. */ *inc_pos = 0; break; } ``` If you remove that, the "Paste" operation eventually completes successfully on `iteration_count == 255762942`, about 25x larger than the current threshold.

Omar commented

2023-03-31 01:42:29 +02:00

Author

It would be nice to get a warning in the UI that the operation failed because of going over the iteration limit. Right now the user gets no feedback whatsoever.

It would be nice to get a warning in the UI that the operation failed because of going over the iteration limit. Right now the user gets no feedback whatsoever.

Chris Blackbourn commented

2023-03-31 02:26:53 +02:00

Member

It would be nice to get a warning in the UI that the operation failed because of going over the iteration limit. Right now the user gets no feedback whatsoever.

I think that's a very reasonable resolution for the 3.5 build.

> It would be nice to get a warning in the UI that the operation failed because of going over the iteration limit. Right now the user gets no feedback whatsoever. I think that's a very reasonable resolution for the 3.5 build.

Pratik Borhade commented

2023-03-31 07:27:14 +02:00

Member

Confirming for now.

Confirming for now.

Pratik Borhade added

and removed

labels 2023-03-31 07:27:34 +02:00

Pratik Borhade referenced this issue

2023-04-04 08:27:55 +02:00

UV: Copy and Paste UVs not work for this islands #106429

Grigoriy Titaev referenced this issue

2023-04-04 09:44:53 +02:00

UV: Copy and Paste UVs not work for this islands #106429

Chris Blackbourn referenced a pull request that will close this issue

2023-04-06 01:06:08 +02:00

Fix #106314: warn user when paste-uvs fails to find match #106606

Chris Blackbourn referenced this issue from a commit

2023-04-06 03:37:11 +02:00

Fix #106314: warn user when paste-uvs fails to find a match

Chris Blackbourn closed this issue

2023-04-06 03:37:12 +02:00

Blender Bot added

and removed

labels 2023-04-06 03:37:14 +02:00

Sign in to join this conversation.

No Label

Animation & Rigging

Asset Browser Project Overview

Automated Testing

Blender Asset Bundle

Dependency Graph

Development Management

EEVEE & Viewport

Images & Movies

Motion Tracking

Nodes & Physics

Pipeline, Assets & IO

Platforms, Builds & Tests

Render & Cycles

Render Pipeline

Sculpt, Paint & Texture

Video Sequencer

Virtual Reality

Blender 2.8 Project

Milestone 1: Basic, Local Asset Browser

Good First Issue

Animation & Rigging

Development Management

EEVEE & Viewport

Nodes & Physics

Pipeline, Assets & IO

Platforms, Builds & Tests

Render & Cycles

Sculpt, Paint & Texture

Needs Info from Developers

Needs Information from User

No Milestone

No project

No Assignees

3 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender#106314

No description provided.