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

Maya schemas cleanup #1610

Merged
merged 6 commits into from
May 31, 2021
Merged

Maya schemas cleanup #1610

merged 6 commits into from
May 31, 2021

Conversation

iLLiCiTiT
Copy link
Member

Changes

  • use advantage of templates where possible
  • converted template_maya_capture to schema_maya_capture as it's used at one place (and that probably won't change)

@iLLiCiTiT iLLiCiTiT self-assigned this May 28, 2021
@iLLiCiTiT iLLiCiTiT requested a review from a team May 28, 2021 17:59
@@ -2156,6 +2156,10 @@ def load_capture_preset(path=None, data=None):
for key in preset['Display Options']:
if key.startswith('background'):
disp_options[key] = preset['Display Options'][key]
disp_options[key][0] = (float(disp_options[key][0])/255)
disp_options[key][1] = (float(disp_options[key][1])/255)
disp_options[key][2] = (float(disp_options[key][2])/255)
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

@@ -2156,6 +2156,10 @@ def load_capture_preset(path=None, data=None):
for key in preset['Display Options']:
if key.startswith('background'):
disp_options[key] = preset['Display Options'][key]
disp_options[key][0] = (float(disp_options[key][0])/255)
disp_options[key][1] = (float(disp_options[key][1])/255)
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

@@ -2156,6 +2156,10 @@ def load_capture_preset(path=None, data=None):
for key in preset['Display Options']:
if key.startswith('background'):
disp_options[key] = preset['Display Options'][key]
disp_options[key][0] = (float(disp_options[key][0])/255)
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

c[0], c[1], c[2])
(float(c[0])/256),
(float(c[1])/256),
(float(c[2])/256)
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

@@ -81,7 +81,10 @@ def process_reference(self, context, name, namespace, options):
if c is not None:
cmds.setAttr(groupName + ".useOutlinerColor", 1)
cmds.setAttr(groupName + ".outlinerColor",
c[0], c[1], c[2])
(float(c[0])/256),
(float(c[1])/256),
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

@@ -62,7 +62,10 @@ def load(self, context, name=None, namespace=None, data=None):
if c is not None:
cmds.setAttr(root + ".useOutlinerColor", 1)
cmds.setAttr(root + ".outlinerColor",
c[0], c[1], c[2])
(float(c[0])/256),
Copy link

Choose a reason for hiding this comment

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

continuation line under-indented for visual indent
missing whitespace around arithmetic operator

c[0], c[1], c[2])
(float(c[0])/256),
(float(c[1])/256),
(float(c[2])/256)
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

@@ -53,7 +53,10 @@ def load(self, context, name, namespace, data):
if c is not None:
cmds.setAttr("{0}.useOutlinerColor".format(group_node), 1)
cmds.setAttr("{0}.outlinerColor".format(group_node),
c[0], c[1], c[2])
(float(c[0])/256),
(float(c[1])/256),
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

@@ -53,7 +53,10 @@ def load(self, context, name, namespace, data):
if c is not None:
cmds.setAttr("{0}.useOutlinerColor".format(group_node), 1)
cmds.setAttr("{0}.outlinerColor".format(group_node),
c[0], c[1], c[2])
(float(c[0])/256),
Copy link

Choose a reason for hiding this comment

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

continuation line under-indented for visual indent
missing whitespace around arithmetic operator

c[0], c[1], c[2])
(float(c[0])/256),
(float(c[1])/256),
(float(c[2])/256)
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

@mkolar mkolar merged commit d0e50fb into develop May 31, 2021
@mkolar mkolar deleted the feature/maya_schemas_cleanup branch May 31, 2021 22:21
@mkolar mkolar added this to the 3.0.0 milestone May 31, 2021
@mkolar mkolar added the type: enhancement Enhancements to existing functionality label May 31, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: enhancement Enhancements to existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants