Check Full BFA shortcuts: Map Clip

Draise's picture
Project: 
Bforartists Tracker
 
 

Check the following code for conflict and resolve some shortcut issues with the following map set:  Map Clip

Current conflict : CTRL + T for clip.track_markers. Currently in BFA that pulls the clip.properties

What should be the alternative?

### Needs Work
# Map Clip
km = kc.keymaps.new('Clip', space_type='CLIP_EDITOR', region_type='WINDOW', modal=False)

kmi = km.keymap_items.new('clip.open', 'O', 'PRESS', alt=True)
kmi = km.keymap_items.new('clip.tools', 'T', 'PRESS')
kmi = km.keymap_items.new('clip.properties', 'T', 'PRESS', ctrl=True)
kmi = km.keymap_items.new('clip.set_solver_keyframe', 'Q', 'PRESS')
kmi_props_setattr(kmi.properties, 'keyframe', 'KEYFRAME_A')
kmi = km.keymap_items.new('clip.set_solver_keyframe', 'E', 'PRESS', shift=True)
kmi_props_setattr(kmi.properties, 'keyframe', 'KEYFRAME_B')
##Added from Blender - by Draise
kmi = km.keymap_items.new('clip.track_markers', 'LEFT_ARROW', 'PRESS', alt=True)
kmi_props_setattr(kmi.properties, 'backwards', True)
kmi_props_setattr(kmi.properties, 'sequence', False)
kmi = km.keymap_items.new('clip.track_markers', 'RIGHT_ARROW', 'PRESS', alt=True)
kmi_props_setattr(kmi.properties, 'backwards', False)
kmi_props_setattr(kmi.properties, 'sequence', False)
#kmi = km.keymap_items.new('clip.track_markers', 'T', 'PRESS', ctrl=True) # COMMENT: conflict with CTRL+T in BFA
#kmi_props_setattr(kmi.properties, 'backwards', False)
#kmi_props_setattr(kmi.properties, 'sequence', True)
kmi = km.keymap_items.new('clip.track_markers', 'T', 'PRESS', shift=True, ctrl=True)
kmi_props_setattr(kmi.properties, 'backwards', True)
kmi_props_setattr(kmi.properties, 'sequence', True)
kmi = km.keymap_items.new('wm.context_toggle_enum', 'TAB', 'PRESS')
kmi_props_setattr(kmi.properties, 'data_path', 'space_data.mode')
kmi_props_setattr(kmi.properties, 'value_1', 'TRACKING')
kmi_props_setattr(kmi.properties, 'value_2', 'MASK')
kmi = km.keymap_items.new('clip.solve_camera', 'S', 'PRESS', shift=True)
kmi = km.keymap_items.new('clip.prefetch', 'P', 'PRESS')

 

 
Status: 
Closed (fixed)
Priority: 
Normal
Category: 
Task
Component: 
Code
Assigned: 
Reporter: 
Created: 
Wed, 01/17/2018 - 17:22
Updated: 
Tue, 03/06/2018 - 15:03

Comments

11
Reiner's picture

Well, you have to fix either the one or the other. It's a conflict I-m so happy

I am of no big help here. I hate the VSE with a passion, and haven't done any motion tracking yet. I would say have a look what is more consistent. Then find a new hotkey for the other item.

Draise's picture

As default BFA is priority, changing the Blender conflict. I would just need a good replacement that would make sense. How about changing them to S? They are good for "Set tracker" to remember and they are used a lot, easier on the hand, and the S is relatively free?  

Reiner's picture

As told, i am no big help here I-m so happy

I'm green with this solution as long as it doesn't introduce yet another conflict Smile

Draise's picture

 
 

### Needs some Work - Practically Done
# Map Clip
km = kc.keymaps.new('Clip', space_type='CLIP_EDITOR', region_type='WINDOW', modal=False)

kmi = km.keymap_items.new('clip.open', 'O', 'PRESS', alt=True)
kmi = km.keymap_items.new('clip.tools', 'T', 'PRESS')
kmi = km.keymap_items.new('clip.properties', 'T', 'PRESS', ctrl=True)
kmi = km.keymap_items.new('clip.set_solver_keyframe', 'Q', 'PRESS') #conflict with Kinorow tools
kmi_props_setattr(kmi.properties, 'keyframe', 'KEYFRAME_A')
kmi = km.keymap_items.new('clip.set_solver_keyframe', 'E', 'PRESS', shift=True)
kmi_props_setattr(kmi.properties, 'keyframe', 'KEYFRAME_B')
##Added from Blender - by Draise
kmi = km.keymap_items.new('clip.track_markers', 'LEFT_ARROW', 'PRESS', alt=True)
kmi_props_setattr(kmi.properties, 'backwards', True)
kmi_props_setattr(kmi.properties, 'sequence', False)
kmi = km.keymap_items.new('clip.track_markers', 'RIGHT_ARROW', 'PRESS', alt=True)
kmi_props_setattr(kmi.properties, 'backwards', False)
kmi_props_setattr(kmi.properties, 'sequence', False)
kmi = km.keymap_items.new('clip.track_markers', 'S', 'PRESS', ctrl=True) # Changed from T to S to avoid conflict with BFA- by Draise
kmi_props_setattr(kmi.properties, 'backwards', False)
kmi_props_setattr(kmi.properties, 'sequence', True)
kmi = km.keymap_items.new('clip.track_markers', 'S', 'PRESS', shift=True, ctrl=True) # Changed from T to S to avoid conflict with BFA - by Draise
kmi_props_setattr(kmi.properties, 'backwards', True)
kmi_props_setattr(kmi.properties, 'sequence', True)
kmi = km.keymap_items.new('wm.context_toggle_enum', 'TAB', 'PRESS')
kmi_props_setattr(kmi.properties, 'data_path', 'space_data.mode')
kmi_props_setattr(kmi.properties, 'value_1', 'TRACKING')
kmi_props_setattr(kmi.properties, 'value_2', 'MASK')
kmi = km.keymap_items.new('clip.solve_camera', 'S', 'PRESS', shift=True) #conflict with Official Pie Menus
kmi = km.keymap_items.new('clip.prefetch', 'P', 'PRESS')

Fixed the T conflicts with S. I also found two new conflicts with two addons from the Blender shortcuts... ridiculous. Kinorow tools and Official Pie Menus.

I will close this and say this needs work and move onto the next map group.

 
Draise's picture

Status: Active » Needs work
Body: Old » New
Draise's picture

### Practically Done
# Map Clip
km = kc.keymaps.new('Clip', space_type='CLIP_EDITOR', region_type='WINDOW', modal=False)

kmi = km.keymap_items.new('clip.open', 'O', 'PRESS', alt=True)
kmi = km.keymap_items.new('clip.tools', 'T', 'PRESS')
kmi = km.keymap_items.new('clip.properties', 'T', 'PRESS', ctrl=True)
kmi = km.keymap_items.new('clip.set_solver_keyframe', 'Q', 'PRESS') #conflict with Kinorow tools and with Official Pie Menus, but feature is in the Pie menu
kmi_props_setattr(kmi.properties, 'keyframe', 'KEYFRAME_A')
kmi = km.keymap_items.new('clip.set_solver_keyframe', 'E', 'PRESS', shift=True) #conflict with Official Pie Menus, but feature is in the Pie menu
kmi_props_setattr(kmi.properties, 'keyframe', 'KEYFRAME_B')
##Added from Blender - by Draise
kmi = km.keymap_items.new('clip.track_markers', 'LEFT_ARROW', 'PRESS', alt=True)
kmi_props_setattr(kmi.properties, 'backwards', True)
kmi_props_setattr(kmi.properties, 'sequence', False)
kmi = km.keymap_items.new('clip.track_markers', 'RIGHT_ARROW', 'PRESS', alt=True)
kmi_props_setattr(kmi.properties, 'backwards', False)
kmi_props_setattr(kmi.properties, 'sequence', False)
kmi = km.keymap_items.new('clip.track_markers', 'S', 'PRESS', ctrl=True) # Changed from T to S to avoid conflict with BFA- by Draise
kmi_props_setattr(kmi.properties, 'backwards', False)
kmi_props_setattr(kmi.properties, 'sequence', True)
kmi = km.keymap_items.new('clip.track_markers', 'S', 'PRESS', shift=True, ctrl=True) # Changed from T to S to avoid conflict with BFA - by Draise
kmi_props_setattr(kmi.properties, 'backwards', True)
kmi_props_setattr(kmi.properties, 'sequence', True)
kmi = km.keymap_items.new('wm.context_toggle_enum', 'TAB', 'PRESS')
kmi_props_setattr(kmi.properties, 'data_path', 'space_data.mode')
kmi_props_setattr(kmi.properties, 'value_1', 'TRACKING')
kmi_props_setattr(kmi.properties, 'value_2', 'MASK')
kmi = km.keymap_items.new('clip.solve_camera', 'S', 'PRESS', shift=True) #conflict with Official Pie Menus
kmi = km.keymap_items.new('clip.prefetch', 'P', 'PRESS')

Finished checking this with funcionanlity. Two BFA shortcuts Q and E both conflict with the Official Pie Menu and Kinoraw addons, but all good. The Official Pie Menu addons have the same feature of said shortcut included in the menu. I will close this. 

Draise's picture

Status: Needs work » Needs review
Draise's picture

Title: Check Full BFA shortcuts: Map Clip - CTRL+T conflict » Check Full BFA shortcuts: Map Clip
Reiner's picture

Okay, what is the task here?

Draise's picture

Ah no task, just needed to double check here and solve the issue and keep it open so you can review. Will close now. 

Draise's picture

Status: Needs review » Closed (fixed)