Alt-M for merging vertices bail out #63781

New Issue

yves bodson · 2019-04-21T12:33:25+02:00

yves bodson commented

2019-04-21 12:33:25 +02:00

System Information
Operating system: Linux-4.18.0-17-generic-x86_64-with-debian-buster-sid 64 Bits
Graphics card: GeForce GTX 1060 6GB/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 390.116

Blender Version
Broken: version: 2.80 (sub 58), branch: master, commit date: 2019-04-21 09:39, hash: c8fc23fdbe
Worked: (optional)

Short description of error
[Please fill out a short description of the error here]
I am using Alt-M to merge vertices on a model.
The process goes as follow : I select two vertices, Alt-M, select merge at center. the merging happens but then when I move the cursor to the next vertices to merge, Blender bails out. Crash.

Exact steps for others to reproduce the error
[Please describe the exact steps needed to reproduce the issue]
[Based on the default startup or an attached .blend file (as simple as possible)]
On the attached blend file, one can select to vertices from the mesh that is in edit mode. do Alt-M and select At center. The merging happen. Do AA to deselect everything, the move the cursor to the next pair of vertices to merge. At that time Blender bails out. Or it may have bailed out on the motion of the curosr.. This is only when doing merging processes. Otherwise I did not experience the crash.

**System Information** Operating system: Linux-4.18.0-17-generic-x86_64-with-debian-buster-sid 64 Bits Graphics card: GeForce GTX 1060 6GB/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 390.116 **Blender Version** Broken: version: 2.80 (sub 58), branch: master, commit date: 2019-04-21 09:39, hash: `c8fc23fdbe` Worked: (optional) **Short description of error** [Please fill out a short description of the error here] I am using Alt-M to merge vertices on a model. The process goes as follow : I select two vertices, Alt-M, select merge at center. the merging happens but then when I move the cursor to the next vertices to merge, Blender bails out. Crash. **Exact steps for others to reproduce the error** [Please describe the exact steps needed to reproduce the issue] [Based on the default startup or an attached .blend file (as simple as possible)] On the attached blend file, one can select to vertices from the mesh that is in edit mode. do Alt-M and select At center. The merging happen. Do AA to deselect everything, the move the cursor to the next pair of vertices to merge. At that time Blender bails out. Or it may have bailed out on the motion of the curosr.. This is only when doing merging processes. Otherwise I did not experience the crash.

yves bodson commented

2019-04-21 12:33:25 +02:00

Added subscriber: @YvesBodson

yves bodson commented

2019-04-21 12:36:37 +02:00

system-info.txt

28 Lilly muguet branches V_04 _01-03.blend

blender28_debug.txt

[system-info.txt](https://archive.blender.org/developer/F6963740/system-info.txt) [28 Lilly muguet branches V_04 _01-03.blend](https://archive.blender.org/developer/F6963741/28_Lilly_muguet_branches_V_04__01-03.blend) [blender28_debug.txt](https://archive.blender.org/developer/F6963743/blender28_debug.txt)

George Vogiatzis commented

2019-04-22 21:42:14 +02:00

Added subscribers: @fclem, @Gvgeo-1

Clément Foucault was assigned by George Vogiatzis

2019-04-22 21:42:14 +02:00

George Vogiatzis commented

2019-04-22 21:42:14 +02:00

@fclem The problem starts with commit 223f7e79f9 (need the file to replicate).

in DRW_cache_free_old_batches(Main *bmain)
in DEG_OBJECT_ITER_BEGIN (depsgraph, ob, iter_flags)

_NODISCARD size_type size() const noexcept
		{	// return length of sequence
		return (static_cast<size_type>(this->_Mylast() - this->_Myfirst()));   <------------------error
		}

I get this error:

 **std::_Vector_alloc<std::_Vec_base_types<DEG::IDNode * __ptr64,std::allocator<DEG::IDNode * __ptr64> > >::_Myfirst**(...) returned 0x10.

@fclem The problem starts with commit 223f7e79f9 (need the file to replicate). in DRW_cache_free_old_batches(Main *bmain) in DEG_OBJECT_ITER_BEGIN (depsgraph, ob, iter_flags) ``` _NODISCARD size_type size() const noexcept { // return length of sequence return (static_cast<size_type>(this->_Mylast() - this->_Myfirst())); <------------------error } ``` I get this error: ``` **std::_Vector_alloc<std::_Vec_base_types<DEG::IDNode * __ptr64,std::allocator<DEG::IDNode * __ptr64> > >::_Myfirst**(...) returned 0x10. ```

George Vogiatzis commented

2019-04-23 07:05:05 +02:00

@fclem Almost forgot.
Because DRW_cache_free_old_batches runs only when there is a screen refresh, you may want to change this line.

  if (ctime % DRW_BATCH_COLLECTION_RATE || ctime == lasttime)
    return;

Right now is a little bit random.

@fclem Almost forgot. Because `DRW_cache_free_old_batches `runs only when there is a screen refresh, you may want to change this line. ``` if (ctime % DRW_BATCH_COLLECTION_RATE || ctime == lasttime) return; ``` Right now is a little bit random.

George Vogiatzis commented

2019-04-24 23:30:39 +02:00

The problem I mention was fixed, with 5a144c797a.

But I noticed this part:

This is only when doing merging processes. Otherwise I did not experience the crash.

@YvesBodson Is your problem fixed? Or still having crashes?

The problem I mention was fixed, with 5a144c797a3b. But I noticed this part: > This is only when doing merging processes. Otherwise I did not experience the crash. @YvesBodson Is your problem fixed? Or still having crashes?

yves bodson commented

2019-04-25 11:29:10 +02:00

Hello
I have downloaded the latest release of 2.8 and did test for the Alt-M behavior. It works fine.
I have tested again with the previous versions and got the issue so the fix is correct.
Thank you for your help and very efficient solution
The issue may be close.
Take care

Hello I have downloaded the latest release of 2.8 and did test for the Alt-M behavior. It works fine. I have tested again with the previous versions and got the issue so the fix is correct. Thank you for your help and very efficient solution The issue may be close. Take care

George Vogiatzis commented

2019-04-25 12:34:46 +02:00

Changed status from 'Open' to: 'Resolved'

George Vogiatzis closed this issue

2019-04-25 12:34:46 +02:00

George Vogiatzis commented

2019-04-25 12:34:46 +02:00

fixed 5a144c797a

@YvesBodson thanks for checking it out.

fixed 5a144c797a3b @YvesBodson thanks for checking it out.

Sign in to join this conversation.