Mocap Tools update 2.81 #69472

New Issue

Brendon Murphy · 2019-09-04T05:06:10+02:00

Brendon Murphy commented

2019-09-04 05:06:10 +02:00

hi, this is the task for the mocap tools update to 2.81 as provided by @SAMtak in D5606
First thing I notice is the PT MT OT could be improved on. MCT_OT_classname would be a good naming convention here. The old way was right in the use of OBJECT_OT_classname, It's cleaned up too much possibly. use MCT_OT_ MCT_PT_ MCT_MT_ would be better I think.
I would like some files so if you could zip up the working addon and provide it here more people will have access without manually applying the patch.
@SAMtak are you prepared to maintain this addon through the 2.8 series and help with future bugs and requests?

hi, this is the task for the mocap tools update to 2.81 as provided by @SAMtak in [D5606](https://archive.blender.org/developer/D5606) First thing I notice is the _PT_ _MT_ _OT_ could be improved on. MCT_OT_classname would be a good naming convention here. The old way was right in the use of OBJECT_OT_classname, It's cleaned up too much possibly. use MCT_OT_ MCT_PT_ MCT_MT_ would be better I think. I would like some files so if you could zip up the working addon and provide it here more people will have access without manually applying the patch. @SAMtak are you prepared to maintain this addon through the 2.8 series and help with future bugs and requests?

Brendon Murphy self-assigned this 2019-09-04 05:06:10 +02:00

Brendon Murphy commented

2019-09-04 05:06:10 +02:00

Added subscribers: @SAMtak, @BrendonMurphy

Brendon Murphy commented

2019-09-06 01:41:09 +02:00

hi, seems like this is a useful and needed addon, I'm concerned about maintenance.
@SAMtak were you prepared to maintain the addon? Any other takers?

hi, seems like this is a useful and needed addon, I'm concerned about maintenance. @SAMtak were you prepared to maintain the addon? Any other takers?

Osamu Takasugi commented

2019-09-09 11:20:56 +02:00

I'm interested mocap addon for a while, I may undertake maintenance If no other person take.

Brendon Murphy commented

2019-09-20 03:12:23 +02:00

hi @SAMtak that would be great, I've committed your patch, I left in the panels closed by default as it's better for the ui space.
I also noted the changes to all the class names. This is not optimal atm. Convention was better in the original.
I would suggest using MOCAP_PT_ for the panels and MOCAP_OT_ for the operators and remove the operator extension at the end. This fits in better with our naming conventions we are trying to encourage.
If you make the changes you can upload a fixed files in zip and I'll check here locally.

hi @SAMtak that would be great, I've committed your patch, I left in the panels closed by default as it's better for the ui space. I also noted the changes to all the class names. This is not optimal atm. Convention was better in the original. I would suggest using MOCAP_PT_ for the panels and MOCAP_OT_ for the operators and remove the operator extension at the end. This fits in better with our naming conventions we are trying to encourage. If you make the changes you can upload a fixed files in zip and I'll check here locally.

Brendon Murphy changed title from ~~Mocap Tools update 2.81 D5606~~ to Mocap Tools update 2.81

2019-09-20 03:13:15 +02:00

Brendon Murphy commented

2019-12-09 02:20:54 +01:00

Changed status from 'Open' to: 'Resolved'

Brendon Murphy closed this issue

2019-12-09 02:20:54 +01:00

Brendon Murphy commented

2019-12-09 02:20:54 +01:00

closing as resolved. no activity, addon is committed in contrib. Currently no plans for advancement until code is cleaned up and a new task submitted by the maintainer.

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

Mocap Tools update 2.81 #69472