Amudtogal / blenderMotionExport

Export addon for scientific use of Blender motion tracking data
MIT License
19 stars 7 forks source link

blender 2.82a python 3.8.3: required parameter to be a keyword #3

Closed xenogenesi closed 4 years ago

xenogenesi commented 4 years ago

Thanks for this addon, I had to make some minor changes to make it work.

--- a/motionExportAddon.py
+++ b/motionExportAddon.py
@@ -62,10 +62,10 @@ class ExportMarkerPanel(bpy.types.Panel):
             x = (active_marker.co.x - other_marker.co.x) * x_size
             y = (active_marker.co.y - other_marker.co.y) * y_size
             fulldistance = np.sqrt(x ** 2 + y ** 2)
-            col.label("Marker distance:")
-            col.label("{0:.2f} ({1:.2f}, {2:.2f})".format(fulldistance, x, y))
+            col.label(text="Marker distance:")
+            col.label(text="{0:.2f} ({1:.2f}, {2:.2f})".format(fulldistance, x, y))
         else:
-            col.label("Select two markers to show distance.")
+            col.label(text="Select two markers to show distance.")

 class ExportTrackingPanel(bpy.types.Panel):
@@ -149,7 +149,7 @@ class ExportDataPanel(bpy.types.Panel):
         layout = self.layout

         col = layout.column(align=True)
-        col.label("Export Path:")
+        col.label(text="Export Path:")
         col.prop(context.scene, "exp_path", text="")
         col.separator()
         col.prop(context.scene, "exp_subdirs")
@@ -159,7 +159,7 @@ class ExportDataPanel(bpy.types.Panel):
         col.prop(context.scene, "frame_end")
         col.operator("scene.fullcliprange")
         col.separator()
-        col.label("Export markers:")
+        col.label(text="Export markers:")
         row = col.row(align=True)
         row.operator("scene.export_marker", text="Selected").selected_only = True
         row.operator("scene.export_marker", text="All")
Amudtogal commented 4 years ago

Hey, thanks for your contribution! The diff does not apply to the blender28 branch, are you sure you were using the blender28 branch? Have you checked this for the latest releaste 1.2? Regards

xenogenesi commented 4 years ago

Oops I hadn't noticed the branch for 2.8, the patch is for the master branch, thanks.

Amudtogal commented 4 years ago

I have updated a new release, use that, should work! :smile: