studio/blender-studio-tools
13
32
Fork 19
Code Issues 21 Pull Requests 4 Projects Releases 1 Activity

Blender Kitsu: Add Import Edit Render Operator for Shots #236

Merged
Nick Alberelli merged 6 commits from TinyNick/blender-studio-pipeline:feature/import_edit_export into main 2024-02-19 21:02:44 +01:00
Conversation 2 Commits 6 Files Changed 9 +280 -121
Nick Alberelli commented 2024-02-19 20:42:28 +01:00
Member
Copy Link

What's Changed?

  • Add Operator to Import Latest Shot's Editorial Render into VSE (similar to how shot builder loads edit renders)
  • Rename All Edit Export Preferences to Edit Render (to match the panel/operator added in studio/blender-studio-pipeline#225)
  • Add Hack to Auto-Fill Edit Render File Patter for Gold
  • Hide Playblast Panel in VSE unless context type is Sequence

Uppdated UI

image

Info/Warning Messages

There are four messages this operator can return with

Info

  • Loaded Latest Editorial Render, 'my_project-edit-v009.mp4' if a render was imported an no old renders were removed
  • Loaded Latest Editorial Render, 'my_project-edit-v010.mp4'. Removed 1 Movie Strips and 1 Audio Strips if a render was imported an some renders were removed

Warning

  • Latest Editorial Render already loaded 'my_project-edit-v010.mp4' if the latest render is already in the VSE
  • No Edit Renders found in '{path_to_edit_render_dir}' using pattern 'my_project-edit-v###.mp4' See Add-On Preferences if the file pattern/file directory is incorrect
## What's Changed? - Add Operator to Import Latest Shot's Editorial Render into VSE (similar to how shot builder loads edit renders) - Rename All Edit Export Preferences to Edit Render (to match the panel/operator added in https://projects.blender.org/studio/blender-studio-pipeline/pulls/225) - Add Hack to Auto-Fill Edit Render File Patter for Gold - Hide Playblast Panel in VSE unless context type is Sequence ## Uppdated UI ![image](/attachments/c6b87aca-46ac-4986-8642-2621672b2185) ## Info/Warning Messages There are four messages this operator can return with #### Info - `Loaded Latest Editorial Render, 'my_project-edit-v009.mp4'` if a render was imported an no old renders were removed - `Loaded Latest Editorial Render, 'my_project-edit-v010.mp4'. Removed 1 Movie Strips and 1 Audio Strips` if a render was imported an some renders were removed #### Warning - `Latest Editorial Render already loaded 'my_project-edit-v010.mp4'` if the latest render is already in the VSE - `No Edit Renders found in '{path_to_edit_render_dir}' using pattern 'my_project-edit-v###.mp4' See Add-On Preferences` if the file pattern/file directory is incorrect
image.png
112 KiB
image.png
36 KiB
image.png
🎉 1
Nick Alberelli added 6 commits 2024-02-19 20:42:32 +01:00
Nick Alberelli changed title from Blender Kitsu: Add Import Edit Render Operator to Blender Kitsu: Add Import Edit Render Operator for Shots 2024-02-19 20:47:14 +01:00
Nick Alberelli merged commit 19fbe6a724 into main 2024-02-19 21:02:43 +01:00
Nick Alberelli deleted branch feature/import_edit_export 2024-02-19 21:02:44 +01:00
Andy Goralczyk commented 2024-02-20 10:01:21 +01:00
Member
Copy Link

This is great! I have one teensy nitpick: I would refer to the video as an "edit export" instead of "render" because that's how we generally refer to it in the studio :)

This is great! I have one teensy nitpick: I would refer to the video as an "edit export" instead of "render" because that's how we generally refer to it in the studio :)
Nick Alberelli commented 2024-02-20 15:11:27 +01:00
Author
Member
Copy Link

@eyecandy Understood, that can be changed, but I would like to get @fsiddi feedback on that because he directed me to use the name Render in this PR and I renamed everything to match that. I am happy to change the name but I would like it to be consistent.

@fsiddi Can we rename all the editorial stuff to export?

So Render & Publish -> Export & Publish
Import Latest Render -> Import Latest Export
etc

@eyecandy Understood, that can be changed, but I would like to get @fsiddi feedback on that because he directed me to use the name Render in this [PR](https://projects.blender.org/studio/blender-studio-pipeline/pulls/225) and I renamed everything to match that. I am happy to change the name but I would like it to be consistent. @fsiddi Can we rename all the editorial stuff to export? So `Render & Publish` -> `Export & Publish` `Import Latest Render` -> `Import Latest Export` etc
👍 2
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
Kind
Breaking
Breaking change that won't be backward compatible

  
Kind
Bug
Something is not working

  
Kind: Community

Issues opened by members of the community / outside the studio

  
Kind
Documentation
Documentation changes

  
Kind
Easy
Simple to complete

  
Kind
Enhancement
Improve existing functionality

  
Kind
Feature
New functionality

  
Kind
Proposal
A proposal or idea that requires feedback

  
Kind
Security
This is security issue

  
Kind
Studio Request
Requested by a user at the Blender-Studio

  
Kind
Testing
Issue or pull request related to testing

  
Priority
Critical
The priority is critical

  
Priority
High
The priority is high

  
Priority
Low
The priority is low

  
Priority
Medium
The priority is medium

  
Reviewed
Confirmed
Issue has been confirmed

  
Reviewed
Duplicate
This issue or pull request already exists

  
Reviewed
Invalid
Invalid issue

  
Reviewed
Won't Fix
This issue won't be fixed

  
Status
Abandoned
Somebody has started to work on this but abandoned work

  
Status
Blocked
Something is blocking this issue or pull request

  
Status
Need More Info
Feedback is required to reproduce issue or to continue work

No Label
Kind
Breaking
Kind
Bug
Kind: Community
Kind
Documentation
Kind
Easy
Kind
Enhancement
Kind
Feature
Kind
Proposal
Kind
Security
Kind
Studio Request
Kind
Testing
Priority
Critical
Priority
High
Priority
Low
Priority
Medium
Reviewed
Confirmed
Reviewed
Duplicate
Reviewed
Invalid
Reviewed
Won't Fix
Status
Abandoned
Status
Blocked
Status
Need More Info
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
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#236
No description provided.
Delete Branch "TinyNick/blender-studio-pipeline:feature/import_edit_export"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?