Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hide/Unhide and Fix in Shortest Path #96

Merged
merged 6 commits into from
Dec 28, 2019
Merged

Conversation

CORPSE-SYS
Copy link
Contributor

No description provided.

Copy link
Owner

@aachman98 aachman98 left a comment

Choose a reason for hiding this comment

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

Awesome! I've mentioned some minor changes in the review if you find the time to add another commit.


def functionality(self):
bpy.ops.mesh.hide(
unselected = self.inputs["Unselected"].default_value,
Copy link
Owner

Choose a reason for hiding this comment

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

No need for ","


def functionality(self):
bpy.ops.mesh.reveal(
select = self.inputs["Select"].default_value,
Copy link
Owner

Choose a reason for hiding this comment

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

same here

@@ -13,8 +13,8 @@ class ScSelectShortestPath(Node, ScSelectionNode):
in_step: BoolProperty(update=ScNode.update_value)
in_distance: BoolProperty(update=ScNode.update_value)
in_fill: BoolProperty(update=ScNode.update_value)
in_nth: IntProperty(default=1, min=1, update=ScNode.update_value)
in_skip: IntProperty(default=1, min=1, update=ScNode.update_value)
in_nth: IntProperty(default=1, min=0, update=ScNode.update_value)
Copy link
Owner

Choose a reason for hiding this comment

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

According to blender docs:
Screenshot from 2019-12-28 10-17-32
"nth" attribute can take minimum value of 1, so a value of 0 will nevertheless be internally to 1.

@@ -31,8 +31,8 @@ def error_condition(self):
return(
super().error_condition()
or (not self.inputs["Edge Tag"].default_value in ['SELECT', 'SEAM', 'SHARP', 'CREASE', 'BEVEL', 'FREESTYLE'])
or self.inputs["Nth Selection"].default_value < 1
or self.inputs["Skip"].default_value < 1
or self.inputs["Nth Selection"].default_value < 0
Copy link
Owner

Choose a reason for hiding this comment

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

I forgot to convert these values using int(). And the function call below also needs to be provided with int arguments for "nth, "skip" and "offset".
If you are editing this commit, then please make these changes too.

@aachman98
Copy link
Owner

More nodes

Fixed Min Nth Selection and set new default value to 0 in "skip" according to blender interface.
Copy link
Contributor Author

@CORPSE-SYS CORPSE-SYS left a comment

Choose a reason for hiding this comment

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

I think I made all the fixes, thanks for your work. :)

@aachman98 aachman98 merged commit 7b4e9b6 into aachman98:master Dec 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants