Page MenuHome
Differential D5358

T67840 fix
AbandonedPublic
Actions

Authored by Brendon Murphy (meta-androcto) on Jul 28 2019, 10:28 PM.

Details

Reviewers
None
Summary

Imperial <-> metric conversion not needed in mesh_edges_length.py it's done somewhere else.

Diff Detail

Event Timeline

Adam Doda (chinpok) created this revision.Jul 28 2019, 10:28 PM
Adam Doda (chinpok) added a reviewer: Add-ons (Community).Jul 29 2019, 9:38 AM
Brendon Murphy (meta-androcto) commandeered this revision.EditedJul 29 2019, 9:57 AM
Brendon Murphy (meta-androcto) added a reviewer: Adam Doda (chinpok).
Brendon Murphy (meta-androcto) added a project: Add-ons (Community).
Brendon Murphy (meta-androcto) added a subscriber: Brendon Murphy (meta-androcto).

hi, I will take a look at this shortly. Is it causing any concerns/errors? I would wait until after release for this and aim at 2.81 unless it's urgent. Thanks for patch

Adam Doda (chinpok) added a comment.Jul 29 2019, 11:27 AM

Hi,

Not urgent for me but right now "Edge Tools -> Set Edge Length" is not usable when unit is set to imperial.

For me this is just a "practice" fix, intro to blender bug fixing. I don't think I'm qualified to prioritize this bug.

Brecht Van Lommel (brecht) removed a reviewer: Add-ons (Community).Jul 30 2019, 12:33 AM
Brendon Murphy (meta-androcto) mentioned this in rBA7008857ef02c: mesh_tools/mesh_edges_length: T67840 D5358.Aug 9 2019, 4:51 AM
Brendon Murphy (meta-androcto) abandoned this revision.Aug 9 2019, 4:53 AM
Brendon Murphy (meta-androcto) removed a reviewer: Adam Doda (chinpok).

tested as valid and committed

Revision Contents
Changeset List

PathSize
mesh_tools/
15 lines

Diff 16641

View Options

mesh_tools/mesh_edges_length.py

Loading...