Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

Global: making collect audio plugin global #3679

Merged
merged 16 commits into from
Sep 8, 2022

Conversation

jakubjezek001
Copy link
Member

@jakubjezek001 jakubjezek001 commented Aug 16, 2022

Brief description

Collect audio plugin could be used by other hosts.

Description

Reviewable videos can be created with included audio tracks if any audio subset is available. Subset name with audio variant can be defined in settings project_settings/global/publish/CollectAudio/audio_subset_name

Testing notes:

  1. standalone publish any image sequence or video on shot with available audio subset.
  2. reviewable should have audio track

@jakubjezek001 jakubjezek001 self-assigned this Aug 16, 2022
@jakubjezek001 jakubjezek001 added the type: feature Larger, user affecting changes and completely new things label Aug 16, 2022
openpype/plugins/publish/collect_audio.py Outdated Show resolved Hide resolved
openpype/plugins/publish/collect_audio.py Outdated Show resolved Hide resolved
openpype/plugins/publish/collect_audio.py Outdated Show resolved Hide resolved
openpype/plugins/publish/collect_audio.py Outdated Show resolved Hide resolved
openpype/plugins/publish/collect_audio.py Outdated Show resolved Hide resolved
@iLLiCiTiT
Copy link
Member

Already removed OcioConfig submodule is added back to repository.

jakubjezek001 and others added 2 commits August 24, 2022 14:14
Co-authored-by: Jakub Trllo <43494761+iLLiCiTiT@users.noreply.github.com>
Co-authored-by: Jakub Trllo <43494761+iLLiCiTiT@users.noreply.github.com>
@@ -3,6 +3,10 @@
"CollectAnatomyInstanceData": {
"follow_workfile_version": false
},
"CollectAudio": {
"enabled": true,
Copy link
Member

@iLLiCiTiT iLLiCiTiT Aug 24, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would turn this plugin off by default as most of studios don't use audio and current state of plugin could do a lot of queries to mongo.

@jrsndl
Copy link
Collaborator

jrsndl commented Sep 7, 2022

I tested on EXR sequence, available audio was added to the reviewable.

Copy link
Member

@iLLiCiTiT iLLiCiTiT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks ok for me.

@jakubjezek001 jakubjezek001 merged commit d630fd6 into develop Sep 8, 2022
@jakubjezek001 jakubjezek001 deleted the feature/global-collect-audio-plugin branch September 8, 2022 14:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: feature Larger, user affecting changes and completely new things
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants