-
Notifications
You must be signed in to change notification settings - Fork 127
updated command to install PySide==1.2.4 #6256
Conversation
…equalizer-integration
…equalizer-integration
…equalizer-integration
…gration' into feature/OP-6032_3dequalizer-integration
…equalizer-integration
…slamEzzat/OpenPype into local/aroma-integrate-3de4
Local/aroma integrate 3de4
FEAT: Create and Extract Warp Footage.
patch("tde4.postQuestionRequester", patched_postQuestionRequester): # noqa: E501 | ||
with open(exporter_path, 'r') as f: | ||
script = f.read() | ||
if not "import tde4" in script: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
test for membership should be 'not in'
|
||
order = pyblish.api.ExtractorOrder | ||
|
||
def process(self, instance = pyblish.api.Instance): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unexpected spaces around keyword / parameter equals
import six | ||
import pyblish.api | ||
import tde4 # noqa: F401 | ||
from openpype.lib import import_filepath |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'openpype.lib.import_filepath' imported but unused
|
||
new_img_width, new_img_height = get_distortion_resolution(staging_dir) | ||
overscan_width = (new_img_width / tde4.getCameraImageWidth(cam)) * 100.0 | ||
overscan_height = (new_img_height / tde4.getCameraImageHeight(cam)) * 100.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
line too long (83 > 79 characters)
thread.join() | ||
|
||
new_img_width, new_img_height = get_distortion_resolution(staging_dir) | ||
overscan_width = (new_img_width / tde4.getCameraImageWidth(cam)) * 100.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
line too long (80 > 79 characters)
import tde4 # noqa: F401 | ||
import threading | ||
from openpype.pipeline import OptionalPyblishPluginMixin, publish | ||
from openpype.hosts.equalizer.api.lib import run_warp4, convert_png, get_distortion_resolution |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
line too long (94 > 79 characters)
else: | ||
overscan_width, overscan_height = 100, 100 | ||
# raise PublishValidationError("You must extract the undistorted footage first to get the overscan") | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
blank line contains whitespace
overscan_width, overscan_height = overscan.values() | ||
else: | ||
overscan_width, overscan_height = 100, 100 | ||
# raise PublishValidationError("You must extract the undistorted footage first to get the overscan") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
line too long (112 > 79 characters)
trailing whitespace
) | ||
from openpype.pipeline import PublishValidationError | ||
|
||
@six.add_metaclass(ABCMeta) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
expected 2 blank lines, found 1
Creator, | ||
OptionalPyblishPluginMixin, | ||
) | ||
from openpype.pipeline import PublishValidationError |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'openpype.pipeline.PublishValidationError' imported but unused
No description provided.