Align XY missing use_alignxy_face_area property #4
Loading…
Reference in New Issue
Block a user
No description provided.
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
The
use_alignxy_face_areaproperty is missing from__init__.py. The property UI then does not get rendered in the Sidebar and the console prints:Suggested solutions
Reinstate the property in the.print_3dscene properties. BTW, I could not find the commit where it got deleted(Alternative)Remove the property row from the Sidebar UI(not my preferred solution), but make the option true per default.I've just found the commit. I think I understand the intention. However, if the property is removed from the UI in the Sidebar, the user risks never seeing it, unless they open up the
Adjust Last Operationmenu.I think this option is very important, and IMO it should default to true. It makes sense for it to be turned on for all the use cases I can think off. Otherwise, the alignment risks being perturbed if there is a large number of very small faces selected together with a few large ones. For instance, if the mesh is from a 3D scan.
Is there a way to show the property on the Sidebar UI without having to include it in the
print_2dscene properties?As an alternative, the option should be on per default. Maybe this is the better way, as I think most users would expect the tool to behave as if it were on, but would not necessarily recognize the need to turn it on, even when the property is clearly visible.
I'm amending my suggestions above.
Note that it is not the only tool that have options in the adjust last operation panel,
Make Manifoldalso has options there, and most tools in Blender are done this way. I do not see the issue with that.Making option enabled by default, is another topic. All my models are always correctly aligned to XY, so I actually never used this tool.
I just now tested it with multiple models of different shapes and topology, and it is indeed resulting in better orientations every single time with the option enabled.
Thank you for catching that! It will be enabled it by default in next release.
Enabling the option is the better way IMO.
Just don't forget to remove the row with the property in
ui.py(line 112) to get rid of therna_uiItemRerror on the console.God dammit! I forgot about ui prop.
Hey, I think
Face Areasis not a descriptive name, do you have a better name in mind for this property? I was thinking aboutWeighted by Face AreaorNormalize by Face AreaorAdjust by Face AreaorAccount for Face Area.I think
Weight by Face AreaorWeight Normals by Face Areawould be more accurate. However, this is a complex concept. It will always be hard to describe it succinctly, and still inform enough to make it clear.But, honestly, I don't really see the point of this needing to be a user-adjustable parameter at all. I can't imagine a use case where you would want to align a model according to some faces and not weight the faces' contribution by their areas.
The only reason I don't suggest removing it right away, is that the person who came up with the tool thought this should be left for the user to decide, so perhaps they see something I don't.
So I think removing it from the sidebar and giving it a sensible default is a good approach. That way, users without deep knowledge don't need to bother with a weird parameter, but those who need to change it (if any) may still have the possibility to do it.