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

Local/seif integrate #6246

Closed

Conversation

seifibrahim32
Copy link

Changelog Description

Paragraphs contain detailed information on the changes made to the product or service, providing an in-depth description of the updates and enhancements. They can be used to explain the reasoning behind the changes, or to highlight the importance of the new features. Paragraphs can often include links to further information or support documentation.

Additional info

Paragraphs of text giving context of additional technical information or code examples.

Testing notes:

  1. start with this step
  2. follow this step

antirotor and others added 30 commits September 13, 2023 14:35
…gration' into feature/OP-6032_3dequalizer-integration
@ynbot ynbot added size/XXL Denotes a PR changes 2500+ lines, ignoring general files host: Maya host: Nuke host: Resolve type: documentation labels Mar 18, 2024
tde4.loadProject(os.path.join(workdir, projects[-1]))


@staticmethod
Copy link

Choose a reason for hiding this comment

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

too many blank lines (2)

# "EqualizerHost._timer", int(heartbeat_interval))


def _set_project(self):
Copy link

Choose a reason for hiding this comment

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

too many blank lines (2)

tde4.setProjectNotes(
re.sub(
CONTEXT_REGEX,
"AYON_CONTEXT::%s::AYON_CONTEXT_END"%update_str,
Copy link

Choose a reason for hiding this comment

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

missing whitespace around modulo operator

return current_filepath


def get_overscan(self):
Copy link

Choose a reason for hiding this comment

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

too many blank lines (2)

def open_workfile(self, filepath):
result = tde4.loadProject(filepath)
if not bool(result):
raise RuntimeError("Failed to open workfile %s."%filepath)
Copy link

Choose a reason for hiding this comment

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

missing whitespace around modulo operator

# set the sequence attributes star/end/step
tde4.setCameraSequenceAttr(
camera, int(version["data"].get("frameStart")),
int(version["data"].get("frameEnd")), 1)
Copy link

Choose a reason for hiding this comment

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

undefined name 'version'


# set the sequence attributes star/end/step
tde4.setCameraSequenceAttr(
camera, int(version["data"].get("frameStart")),
Copy link

Choose a reason for hiding this comment

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

undefined name 'version'


image_dir, image_path = os.path.split(file_path)
image_name, id, image_format = image_path.split(".")
pattern = ".".join([image_name, len(id)*"#", image_format])
Copy link

Choose a reason for hiding this comment

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

missing whitespace around arithmetic operator

tde4.getCameraName(c) == container["namespace"]
][0]
except IndexError:
self.log.error('Cannot find camera {}'.format(container["namespace"]))
Copy link

Choose a reason for hiding this comment

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

line too long (82 > 79 characters)

tde4.setLensFocalLength(lens_id, float(camera_focal[0]))


def update(self, container, representation):
Copy link

Choose a reason for hiding this comment

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

too many blank lines (2)

@ynbot ynbot added this to the next-patch milestone Mar 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
host: Maya host: Nuke host: Resolve size/XXL Denotes a PR changes 2500+ lines, ignoring general files type: documentation
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants