CAD_Sketcher icon indicating copy to clipboard operation
CAD_Sketcher copied to clipboard

Entity conversion resolution

Open hlorus opened this issue 3 years ago • 5 comments

Right now converted meshes approximate curves with a given resolution. Ideally this resolution could be configured as users might want more detail on the converted mesh.

Since mesh converting is currently based on the bezier converter this simply boils down to setting the correct values in the bezier datablock before the to_mesh() method is called. Note that setting this property by the user doesn't work because the bezier object is temporary an gets recreated every time conversion happens.

This setting could probably be put in one of a few different scopes e.g. one setting for the addon/ per sketch/ per spline. For now it should be sufficient to simply have it per sketch.

hlorus avatar Apr 27 '22 09:04 hlorus

What about multires support too?

SupinePandora43 avatar May 02 '22 14:05 SupinePandora43

Hello! Sorry if this should be obvious, but I don't really understand what this feature is about ; could it be possible to add some additional information, given this is tagged as "good first issue"? I'm thinking "current behavior", "expected behavior", and some pointers as to where in the codebase the change should be made?

It would be awesome if such extra information could be added to other issues with the "good first issue" tag too, please :-) thanks in advance!

Cheaterman avatar May 08 '22 11:05 Cheaterman

Hello! Sorry if this should be obvious, but I don't really understand what this feature is about ; could it be possible to add some additional information, given this is tagged as "good first issue"? I'm thinking "current behavior", "expected behavior", and some pointers as to where in the codebase the change should be made?

It would be awesome if such extra information could be added to other issues with the "good first issue" tag too, please :-) thanks in advance!

There's no way to specify how many polygons sketch-to-mesh result should have (i think)

SupinePandora43 avatar May 08 '22 12:05 SupinePandora43

Hello! Sorry if this should be obvious, but I don't really understand what this feature is about ; could it be possible to add some additional information, given this is tagged as "good first issue"? I'm thinking "current behavior", "expected behavior", and some pointers as to where in the codebase the change should be made?

It would be awesome if such extra information could be added to other issues with the "good first issue" tag too, please :-) thanks in advance!

Absolutely agree with that, thanks for pointing out! :)

hlorus avatar May 09 '22 05:05 hlorus

IMO it's better to have this solved by a generic system rather than hardcoding properties that the user might want to change.

There is already an attempt to solve this, see #200.

hlorus avatar Jun 22 '22 14:06 hlorus

Closed by fd3b9e3cc30dbbac5a27f3ed840ff1487b40621c

hlorus avatar Aug 30 '22 12:08 hlorus