Fit Shape теперь учитывает размеры меша
#806
Conversation
igelbox
force-pushed
the
issues/803/wrong-bone-shape-fit
branch
from
6abb6fa
to
d550eeb
Compare
igelbox
force-pushed
the
issues/803/wrong-bone-shape-fit
branch
from
d550eeb
to
cca36d8
Compare
|
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## develop #806 +/- ##
========================================
Coverage 78.88% 78.88%
========================================
Files 226 226
Lines 20945 20947 +2
Branches 3506 3507 +1
========================================
+ Hits 16523 16525 +2
Misses 3516 3516
Partials 906 906 ☔ View full report in Codecov by Sentry. |
| @@ -137,7 +137,7 @@ def _bone_objects(bone): | |||
| break | |||
|
|
|||
|
|
|||
| def bone_vertices(bone): | |||
| def bone_vertices(bone, in_world_coordinates=False): | |||
There was a problem hiding this comment.
Я бы сделал True, но оно где-то в OGF используется, а мне так не хочется лезть в это древнее зло)
| @@ -172,6 +173,17 @@ def assertReportsNotContains(self, report_type=None, re_message=None): | |||
| report | |||
| )) | |||
|
|
|||
| def assertAlmostEqualV(self, first, second, delta, msg=None): | |||
| class V: | |||
There was a problem hiding this comment.
Может однобуквенные идентификаторы лучше не создавать? И в таких именах assertAlmostEqualV писать в конце подробнее: assertAlmostEqualVec?
И поворот, и прочие аффинные преобразования.
Думаю, работать в world-системе-координат - проще для понимания => багов будет меньше.
[fixes #803]