Move Anim_Setup module into Blender_Kitsu #5

Merged
Nick Alberelli merged 27 commits from :feature/merge_anim_setup_into_blender_kitsu into master 2023-04-05 17:38:41 +02:00
Member

Closes: TinyNick/blender-studio-tools#10

Issue

Several functions within Anim_Setup module should be automatically run during shot_builder process. This was requested by several people in the studio.

Whats Changed?

  • Load Shot data from central Blender_Kitsu.Gazu module
  • Load latest Editorial Export into VSE and shift strips to correct time
    • Allow user to offset this video with integer in preferences
  • Setup Workspaces for Animation
  • Create Action with Fake user on all Armatures (based on naming conventions)
  • Set correct Frame Range per Kitsu Shot data
  • Allow User to run python commands after running shot_builder
  • Add VSE to Animation Workspace to show user Editorial Export
  • Save File only at the end of the operation

Features Skipped from Anim_Setup Module

- [ ] Import camera action from the previs file
- [ ] Import actions for found assets from previs file
- [ ] Shift animation of camera and asset actions to start at layout cut in

Features skipped because they rely on a previz.blend file that doesn't exist in the studio anymore. Shots are arranged with camera markers with is another legacy feature.

Notes

This merge focuses add the additional automation steps to Blender_Kitsu's shot_builder. These are steps requested by users in the studio, and should be done soon. It does not re-factor or change the prexisting logic of the shot_builder. A refactor will be necessary to make this operator more streamlined. TinyNick/blender-studio-tools#26

Closes: https://projects.blender.org/TinyNick/blender-studio-tools/issues/10 # Issue Several functions within Anim_Setup module should be automatically run during shot_builder process. This was requested by several people in the studio. # Whats Changed? - Load Shot data from central Blender_Kitsu.Gazu module - Load latest Editorial Export into VSE and shift strips to correct time - Allow user to offset this video with integer in preferences - Setup Workspaces for Animation - Create Action with Fake user on all Armatures (based on naming conventions) - Set correct Frame Range per Kitsu Shot data - Allow User to run python commands after running shot_builder - Add VSE to Animation Workspace to show user Editorial Export - Save File only at the end of the operation # Features Skipped from [Anim_Setup](https://projects.blender.org/TinyNick/blender-studio-tools/commits/branch/master/anim_setup) Module ~~- [ ] Import camera action from the previs file~~ ~~- [ ] Import actions for found assets from previs file~~ ~~- [ ] Shift animation of camera and asset actions to start at layout cut in~~ *Features skipped because they rely on a previz.blend file that doesn't exist in the studio anymore. Shots are arranged with camera markers with is another legacy feature.* ### Notes This merge focuses add the additional automation steps to Blender_Kitsu's shot_builder. These are steps requested by users in the studio, and should be done soon. It does not re-factor or change the prexisting logic of the shot_builder. A refactor will be necessary to make this operator more streamlined. https://projects.blender.org/TinyNick/blender-studio-tools/issues/26
Nick Alberelli added 26 commits 2023-04-04 20:14:48 +02:00
- Move Shot_builder into Blender_Kitsu
 - Changes from older branch
- add operator file
- add registration to __init__.py
- Adds storyboard ref from editorial inside scene
- Re-use logic from anim_setup
- import video from editorial folder during shot_build
- rename to 'Editorial Export Directory'
 - add description with hint to current editorial export path
- Find all Armature Objects
- Set Action name to match blender-studio convention
- Set action to fake user
- add edit_export_file_pattern to prefs
 - expose file pattern in addon preferences
 - check is_editorial_dir_valid() with function to prefs
 - Cancel shot_builder if edit_export_dir or edit_export_file_pattern is invalid
- Clear "TODO EXPOSE INT IN ADDON PREFERENCES"
 - Move shot_builder addon prefs above misc
 - Expose Start Frame Offset in addon pref
 - Nest Start Frame under 'show advanced'
 - Set frame range via ''shot_builder_frame_offset"
- expose armature prefix as string in addon pref
 - expose action prefix as string in addon pref
 - give both detailed descriptions
 - use above string props in shot_builder; intilize armatures
- Move anim_setup module to shot_builder
 - Refactor anim_setup functions into core functions that are called by operators, to easily re-use functions in shot_builder
 - Warn user before execution if editorial export directory is invalid (don't used prop update) use @property decorator instead
 - Centralize logic to check if path is valid in anim_setup module
 - remove duplicate code
 - replace bpy.ops.anim_setup with new function calls in shot_builder
 - use frame offset in editorial_export_get_latest
 - only report if workspace was deleted once in `ANIM_SETUP_OT_setup_workspaces`
- Add animation_workspace_vse_area_add() function
- Comment out function call in operator
- Add TODO to fix UI dependant call
- main shot_builder function to modal
- restore original formula to offset sequence strip
- restore original variables to offset sequence strip
- move functions related to importing references into editorial module from the anim_setup module
- nest editorial under shot_builder module
- move report outside of core function
- return none if no shot is found but server is connected
- don't save in the middle of the function call
- extract save logic and move to main operator
- build gazu context manually without relying on file path context
- use gazu.shot in editorial standalone operator
- reorganize main operator
- Add option to skip save
 - Add feedback if file already exists error
 - Move logic back to save_fily.py and make it callable from other functions
- create text box that user can supply commands into incase there is customization for the production to be done
- Create Editorial Export Offset property
 - Use property when placing VSE reference
 - Expose Property in UI
commit f85db92325
Author: Nick Alberelli <tinynick@noreply.localhost>
Date:   Tue Apr 4 16:43:32 2023 +0200

    Move Shot Builder into Blender Kitsu

    Closes: TinyNick/blender-studio-tools#2

    The goal of this merge is to re-use to 'Blender Kitsu' addon's gazu based login/api functions in the 'shot builder' addon.

    **What's Changed**
     - Move Shot_builder files to Blender Kitsu Addon
     - Remove Legacy Shot_builder preferences
     - Move Shot_builder properties into Blender Kitsu
     - Use Gazu within shot_builder's kitsu connector
     - Restore Shot Builder Documentation

    Co-authored-by: TinyNick <nick@blender.org>
    Reviewed-on: #3

commit cbfc016ea3
Author: Demeter Dzadik <demeter@blender.org>
Date:   Tue Apr 4 15:37:42 2023 +0200

    AP: reset_armature_pose() now resets booleans

commit 4568ed7f35
Author: Demeter Dzadik <demeter@blender.org>
Date:   Tue Apr 4 15:36:41 2023 +0200

    AssetPipeline: Nudge GeoNode modifiers

    This is necessary because without it, sometimes GeoNode modifier
    inputs just disappear...
Nick Alberelli force-pushed feature/merge_anim_setup_into_blender_kitsu from 85dba24d3a to 4a5cd67003 2023-04-04 20:20:53 +02:00 Compare
Nick Alberelli added 1 commit 2023-04-04 20:24:53 +02:00
Nick Alberelli changed title from WIP: feature/merge_anim_setup_into_blender_kitsu to Merge `Anim_Setup` into `Blender_Kitsu` 2023-04-04 20:26:22 +02:00
Nick Alberelli requested review from Sebastian Parborg 2023-04-04 20:36:12 +02:00
Sebastian Parborg approved these changes 2023-04-05 11:28:26 +02:00
Sebastian Parborg left a comment
Member

Nice code reduction!
LGTM

Nice code reduction! LGTM
Nick Alberelli changed title from Merge `Anim_Setup` into `Blender_Kitsu` to Move `Anim_Setup` module into `Blender_Kitsu` 2023-04-05 17:38:13 +02:00
Nick Alberelli changed title from Move `Anim_Setup` module into `Blender_Kitsu` to Move `Anim_Setup` module into `Blender_Kitsu` 2023-04-05 17:38:18 +02:00
Nick Alberelli merged commit 97aadc9ee4 into master 2023-04-05 17:38:41 +02:00
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: studio/blender-studio-tools#5
No description provided.