Mikhail Rachinskiy MikhailRachinskiy
  • Joined on 2010-02-17
Mikhail Rachinskiy commented on pull request blender/blender-addons#105233 2024-03-18 19:35:08 +01:00
Update object_print3d_utils/operators.py

It should be something like this:

git checkout mybranch
git merge origin/main
git push
Mikhail Rachinskiy commented on pull request blender/blender-addons#105233 2024-03-18 12:53:46 +01:00
Update object_print3d_utils/operators.py

Hi, could you please bring this branch up to date and remove redundant comment? I will merge it right away.

Mikhail Rachinskiy pushed to main at blender/blender-addons 2024-03-18 12:35:10 +01:00
eb6f507f66 3D-Print: Cleanup unused property after recent merge
Mikhail Rachinskiy pushed to main at blender/blender-addons 2024-03-18 12:24:31 +01:00
f5205cf91a 3D-Print: Add Hollow tool
Mikhail Rachinskiy merged pull request blender/blender-addons#105194 2024-03-18 12:24:31 +01:00
3D Print Toolbox: Add hollow out
Mikhail Rachinskiy approved blender/blender-addons#105194 2024-03-18 12:09:52 +01:00
3D Print Toolbox: Add hollow out

OK then, it is time for merge.

Mikhail Rachinskiy commented on pull request blender/blender-addons#105194 2024-03-18 11:01:45 +01:00
3D Print Toolbox: Add hollow out

In my testing I found that this: If min value magnitude less then 2 * voxel size was not a reliable way to test.

  • Suzanne model unmodified
  • Inside offset 0.1
  • Voxel size 0.3

I…

Mikhail Rachinskiy commented on pull request blender/blender-addons#105194 2024-03-17 11:36:46 +01:00
3D Print Toolbox: Add hollow out

OK it's ready for merge. There is just one more thing that I wanted to discuss with you.

I noticed there are two reasons hollow gives empty result (object with no mesh):

  1. Offset value is…
Mikhail Rachinskiy commented on pull request blender/blender-addons#105194 2024-03-17 06:07:03 +01:00
3D Print Toolbox: Add hollow out

Enum values are supposed to be in UPPERCASE

Mikhail Rachinskiy commented on pull request blender/blender-addons#105194 2024-03-17 06:03:22 +01:00
3D Print Toolbox: Add hollow out

You did not expand enum property so it currently shows as dropdown list.

Add draw method to the operator, right before execute method and manually compose UI:

def draw(self,
Mikhail Rachinskiy commented on pull request blender/blender-addons#105194 2024-03-17 05:33:46 +01:00
3D Print Toolbox: Add hollow out

Adding presets was is just a suggestion,

Mikhail Rachinskiy suggested changes for blender/blender-addons#105194 2024-03-14 08:02:39 +01:00
3D Print Toolbox: Add hollow out

Nice to have you back, meanwhile I got the flu and currently coughing my lungs out, but it's getting better.

Mikhail Rachinskiy commented on pull request blender/blender-addons#105194 2024-03-14 08:02:39 +01:00
3D Print Toolbox: Add hollow out

default=1

Mikhail Rachinskiy commented on pull request blender/blender-addons#105194 2024-03-14 08:02:39 +01:00
3D Print Toolbox: Add hollow out

I think that simple checkbox called inside is not user friendly, it is fine with on/off options, but with something more complex it's better to make all available options visible right away (even when there is only two options).

Mikhail Rachinskiy commented on pull request blender/blender-addons#105194 2024-03-14 08:02:39 +01:00
3D Print Toolbox: Add hollow out

The scene settings have to go, it's hidden behavior and is really convoluted without clear benefit, and for reusing settings it is better to use presets.

Mikhail Rachinskiy commented on pull request blender/blender-addons#105194 2024-02-29 00:22:13 +01:00
3D Print Toolbox: Add hollow out

I'm not suggesting removing the bi-directional offset functionality, just flip the sign, so positive would mean inward and negative outward, similar to solidify modifier.

The only software that I…

Mikhail Rachinskiy commented on pull request blender/blender-addons#105194 2024-02-27 21:37:13 +01:00
3D Print Toolbox: Add hollow out

The name of the panel should be changed from Transform to Edit, Hollow tool should be placed first (no additional label needed).

Mikhail Rachinskiy commented on pull request blender/blender-addons#105194 2024-02-27 21:37:13 +01:00
3D Print Toolbox: Add hollow out

Double newline only allowed in-between class and function definitions, everything inside function body should be separated by 1 line max.

Mikhail Rachinskiy commented on pull request blender/blender-addons#105194 2024-02-27 21:37:13 +01:00
3D Print Toolbox: Add hollow out

These comments are redundant, it's obvious why you flip normals, why you move new object to a location of current object, or why you remove mesh (you could name it mesh_temp to emphasize intent).

Mikhail Rachinskiy commented on pull request blender/blender-addons#105194 2024-02-27 21:37:13 +01:00
3D Print Toolbox: Add hollow out
  • Operator should be called Hollow, it is short and widely accepted terminology.