* OP-6345 - updated logic to return path and comp for FootageItem
Used later to check existance of file in published comps
* OP-6345 - added validator if footage files exist
Comp could contain multiple FootageItems, eg imported file(s). If file is missing render triggered by jsx fails silently.
* OP-6345 - updated extension
* OP-6345 - small updates after testing
* OP-6345 - fix - handle Solid Footage items
JSX failed silently on Solid item as it doesn't have any `.file`
* OP-6345 - enhance documentation
* OP-6345 - remove optionality
This plugin shouldn't be optional as when needed and skipped it result in really weird behavior.
* OP-6345 - updated documentation
Added missing plugins.
* OP-6345 - missed functionality for optionality
* OP-6345 - removed unneeded import
* OP-6763 - refresh all visible for Flatten image
Previously newly added layers were missing.
* OP-6763 - added explicit image collector
Creator was adding 'layer' metadata from workfile only during collect_instances, it was missing for newly added layers. This should be cleaner approach
* OP-6763 - removed unnecessary method overwrite
Creator is not adding layer to instance, separate collector created.
* OP-6763 - cleanup of names
Was failing when template for subset name for image family contained {layer}
* OP-6763 - cleanup, removed adding layer metadata
Separate collector created, cleaner.
Fixed propagation of mark_for_review
* OP-6763 - using members instead of layer data
Members should be more reliable.
* OP-6763 - updated image from Settings
Explicit subset name template was removed some time ago as confusing.
* OP-6763 - added explicit local plugin
Automated plugin has different logic, local would need to handle if auto_image is disabled by artist
* OP-6763 - Hound
* OP-6345 - fix - review for image family
Image family instance contained flattened content. Now it reuses previously extracted file without need to re-extract.
* OP-6358 - allow not creation of group for reference loader in Maya
Creation of wrapping group could be controlled by checkbox in ReferenceLoader options.
* OP-6358 - group name could be empty in Settings
This enables default behavior of not creating wrapping group without need of artists unchecking `Group imported assets`.
* OP-6358 - changed to safer logic
Stripping of | was weird and potentially dangerous (collision of names), this logic should be safer.
* prepared helper functions for custom settings apply method
* publish plugin can have 'settings_category' attribute to define settings category
* Better 'settings_category' comment
Co-authored-by: Roy Nieterau <roy_nieterau@hotmail.com>
* fix trailing spaces
* added more information about pyblish plugins to dev docs
---------
Co-authored-by: Roy Nieterau <roy_nieterau@hotmail.com>
* OP-5660 - adding menu buttons to Set frame range in AE
* OP-5660 - refactored location of scripts
set_settings should be in lib as it is used elsewhere, but launch_logic and lib created circular dependency.
Moved main to launch logic as it is actually handling launching.
* OP-5660 - added set_settings to creator
When instance gets created, set frame range and resolution from DB.
* OP-5660 - minor fix
* OP-5660 - updated extension zip
* OP-5660 - updated documentation
* OP-5660 - fixed missing exception
* OP-5660 - fixed argument
* OP-5560 - fix imports
* OP-5660 - updated extension
* OP-5660 - add js alert message for buttons
* OP-5660 - repacked extension
Without Anastasyi showed success, but extension wasn't loaded.
* OP-5660 - make log message nicer
* OP-5660 - added log if workfile not saved
* OP-5660 - provide defaults to limit None exception
* OP-5660 - updated error message
* Fix: Locally copied version of last published workfile is not incremented
* fix subset first match
* correct anatomy name
* Fix typo and linting
* keep source filepath for further path conformation
* fetch also input dependencies of workfile
* required changes
* lint
* fix case only one subset
* Enhancement: copy last workfile as reusable methods (#6)
* Enhancement: copy last published workfile as reusable methods (WiP)
* Added get_host_extensions method, added subset_id and las_version_doc access, added optional arguments to get_last_published_workfile
* Plugged in the new methods + minor changes
* Added docstrings, last workfile optional argument, and removed unused code
* Using new implementation to get local workfile path. Warning: It adds an extra dot to the extension which I need to fix
* Refactoring and fixed double dots
* Added match subset_id and get representation method, plus clan up
* Removed unused vars
* Fixed some rebasing errors
* delinted unchanged code and renamed get_representation into get_representation_with_task
* This time it's really delinted, I hope...
* Update openpype/modules/sync_server/sync_server.py
reprenation isn't the right spelling (:
Co-authored-by: Félix David <felixg.david@gmail.com>
* Changes based on reviews
* Fixed non imperative docstring and missing space
* Fixed another non imperative docstring
* Update openpype/modules/sync_server/sync_server.py
Fixed typo
Co-authored-by: Félix David <felixg.david@gmail.com>
Co-authored-by: Hayley GUILLOT <hayleyguillot@outlook.com>
Co-authored-by: Félix David <felixg.david@gmail.com>
* Fix: syntax error
* fix single subset case
* Restore sync server enabled test in hook
* Python2 syntax
* renaming and missing key case handling
* Fix local workfile overwritten on update in some cases (#7)
* Fix: Local workfile overwrite when local version number is higher than published workfile version number (WiP)
* Changed regex search, clean up
* Readded mistakenly removed newline
* lint
* remove anticipated functions for cleaner PR
* remove funcs from entities.py
* change to get_last_workfile_with_version
* clean
* Update openpype/modules/sync_server/sync_server.py
Co-authored-by: Jakub Trllo <43494761+iLLiCiTiT@users.noreply.github.com>
* removed get_last_published_workfile_path
* moved hook to sync server module
* fix lint
* Refactor - download only if not present
* Refactor - change to list instead of set
* Refactor - removing unnecessary code
last_published_workfile_path must exists or we wouldn't get there. Use version only from that.
* Refactor - removing unnecessary imports
* Added check for max fail tries
* Refactor - cleaned up how to get last workfile
* Updated docstrings
* Remove unused imports
Co-authored-by: Félix David <felixg.david@gmail.com>
* OP-5466 - run this on more DCC
* Updated documentation
* Fix - handle hero versions
Skip hero versions, look only for versioned published to get max version id.
* Hound
* Refactor - simplified download_last_published_workfile
Logic should be in pre hook
* Skip if no profile found
* Removed unwanted import
* Use collected project_doc
Co-authored-by: Jakub Trllo <43494761+iLLiCiTiT@users.noreply.github.com>
* Use cached project_settings
Co-authored-by: Jakub Trllo <43494761+iLLiCiTiT@users.noreply.github.com>
---------
Co-authored-by: Félix David <felixg.david@gmail.com>
Co-authored-by: Sharkitty <81646000+Sharkitty@users.noreply.github.com>
Co-authored-by: Hayley GUILLOT <hayleyguillot@outlook.com>
Co-authored-by: Jakub Trllo <43494761+iLLiCiTiT@users.noreply.github.com>
Co-authored-by: Jakub Ježek <jakubjezek001@gmail.com>
* OP-5656 - added auto creator for review in PS
Review instance should be togglable.
Review instance needs to be created for non publisher based workflows.
* OP-5656 - refactored names
* OP-5656 - refactored names
* OP-5656 - new auto creator for flat image
In old version flat image was created if no instances were created. Explicit auto creator added for clarity.
Standardization of state of plugins
* OP-5656 - updated according to auto image creator
Subset template should be used from autocreator and not be separate.
* OP-5656 - fix proper creator name
* OP-5656 - fix log message
* OP-5656 - fix use enable state
* OP-5656 - fix formatting
* OP-5656 - add review toggle to image instance
For special cases where each image should have separate review.
* OP-5656 - fix description
* OP-5656 - fix not present asset and task in instance context
* OP-5656 - refactor - both auto creators should use same class
Provided separate description.
* OP-5656 - fix - propagate review to families
Image and auto image could have now review flag.
Bottom logic is only for Webpublisher.
* OP-5656 - fix - rename review files to avaid collision
Image family produces jpg and png, jpg review would clash with name. It should be replaced by 'jpg_jpg'.
* OP-5656 - fix - limit additional auto created only on WP
In artist based publishing auto image would be created by auto creator (if enabled). Artist might want to disable image creation.
* OP-5656 - added mark_for_review flag to Publish tab
* OP-5656 - fixes for auto creator
* OP-5656 - fixe - outputDef not needed
outputDef should contain dict of output definition. In PS it doesn't make sense as it has separate extract_review without output definitions.
* OP-5656 - added persistency of changes to auto creators
Changes as enabling/disabling, changing review flag should persist.
* OP-5656 - added documentation for admins
* OP-5656 - added link to new documentation for admins
* OP-5656 - Hound
* OP-5656 - Hound
* OP-5656 - fix shared families list
* OP-5656 - added default variant for review and workfile creator
For workfile Main was default variant, "" was for review.
* OP-5656 - fix - use values from Settings
* OP-5656 - fix - use original name of review for main review family
outputName cannot be in repre or file would have ..._jpg.jpg
* OP-5656 - refactor - standardized settings
Active by default denotes if created instance is active (eg. publishable) when created.
* OP-5656 - fixes for skipping collecting auto_image
data["ids"] are necessary for extracting. Members are physical layers in image, ids are "virtual" items, won't get grouped into real image instance.
* OP-5656 - reworked auto collectors
This allows to use automatic test for proper testing.
* OP-5656 - added automatic tests
* OP-5656 - fixes for auto collectors
* OP-5656 - removed unnecessary collector
Logic moved to auto collectors.
* OP-5656 - Hound
