[Bf-blender-cvs] [7d5d27ed52f] pygpu_extensions: Cleanup: delete comments with test index

Habib Gahbiche noreply at git.blender.org
Fri Feb 12 16:17:19 CET 2021


Commit: 7d5d27ed52f125066ec8913a30090859151906b5
Author: Habib Gahbiche
Date:   Thu Feb 11 22:56:51 2021 +0530
Branches: pygpu_extensions
https://developer.blender.org/rB7d5d27ed52f125066ec8913a30090859151906b5

Cleanup: delete comments with test index

Tests are not identified with indexes, so no need to maintain comments with indexes anymore

Reviewed By: calra, mont29

Differential Revision: https://developer.blender.org/D10277

===================================================================

M	tests/python/modules/mesh_test.py
M	tests/python/operators.py

===================================================================

diff --git a/tests/python/modules/mesh_test.py b/tests/python/modules/mesh_test.py
index b1dadfdb5af..a58b803b22d 100644
--- a/tests/python/modules/mesh_test.py
+++ b/tests/python/modules/mesh_test.py
@@ -580,7 +580,7 @@ class MeshTest:
         compare_success = (compare_result == 'Same')
 
         # Also check if invalid geometry (which is never expected) had to be corrected...
-        validation_success = evaluated_test_mesh.validate(verbose=True) == False
+        validation_success = not evaluated_test_mesh.validate(verbose=True)
 
         if compare_success and validation_success:
             if self.verbose:
diff --git a/tests/python/operators.py b/tests/python/operators.py
index 93cdfebab7b..db7e70c09c4 100644
--- a/tests/python/operators.py
+++ b/tests/python/operators.py
@@ -37,7 +37,6 @@ MONKEY_LOOP_EDGE = {131, 278, 299, 305, 307, 334, 337, 359, 384, 396, 399, 412,
 
 def main():
     tests = [
-        #### 0
         # bisect
         MeshTest("CubeBisect", "testCubeBisect", "expectedCubeBisect",
                  [OperatorSpecEditMode("bisect",
@@ -57,7 +56,6 @@ def main():
                  [OperatorSpecEditMode("decimate",
                                        {"ratio": 0.1}, "FACE", {i for i in range(500)})]),
 
-        ### 4
         # delete
         MeshTest("CubeDeleteVertices", "testCubeDeleteVertices", "expectedCubeDeleteVertices",
                  [OperatorSpecEditMode("delete", {}, "VERT", {3})]),
@@ -75,7 +73,6 @@ def main():
                  "expectedMonkeyDeleteEdgeLoopEdges",
                  [OperatorSpecEditMode("delete_edgeloop", {}, "EDGE", MONKEY_LOOP_EDGE)]),
 
-        ### 9
         # delete loose
         MeshTest("CubeDeleteLooseVertices", "testCubeDeleteLooseVertices",
                  "expectedCubeDeleteLooseVertices",
@@ -98,7 +95,6 @@ def main():
                  "expectedCubeDissolveDegenerate",
                  [OperatorSpecEditMode("dissolve_degenerate", {}, "VERT", {i for i in range(8)})]),
 
-        ### 13
         # dissolve edges
         MeshTest("CylinderDissolveEdges", "testCylinderDissolveEdges", "expectedCylinderDissolveEdges",
                  [OperatorSpecEditMode("dissolve_edges", {}, "EDGE", {0, 5, 6, 9})]),
@@ -107,7 +103,6 @@ def main():
         MeshTest("CubeDissolveFaces", "testCubeDissolveFaces", "expectedCubeDissolveFaces",
                  [OperatorSpecEditMode("dissolve_faces", {}, "VERT", {5, 34, 47, 49, 83, 91, 95})]),
 
-        ### 15
         # dissolve verts
         MeshTest("CubeDissolveVerts", "testCubeDissolveVerts", "expectedCubeDissolveVerts",
                  [OperatorSpecEditMode("dissolve_verts", {}, "VERT", {16, 20, 22, 23, 25})]),
@@ -124,7 +119,6 @@ def main():
         MeshTest("ConeDuplicateEdges", "testConeDuplicateEdges", "expectedConeDuplicateEdges",
                  [OperatorSpecEditMode("duplicate", {}, "EDGE", {i for i in range(64)})]),
 
-        ### 20
         # edge collapse
         MeshTest("CylinderEdgeCollapse", "testCylinderEdgeCollapse", "expectedCylinderEdgeCollapse",
                  [OperatorSpecEditMode("edge_collapse", {}, "EDGE", {1, 9, 4})]),
@@ -143,7 +137,6 @@ def main():
         MeshTest("CubeEdgeSplit", "testCubeEdgeSplit", "expectedCubeEdgeSplit",
                  [OperatorSpecEditMode("edge_split", {}, "EDGE", {2, 5, 8, 11, 14, 17, 20, 23})]),
 
-        ### 25
         # face make planar
         MeshTest("MonkeyFaceMakePlanar", "testMonkeyFaceMakePlanar",
                  "expectedMonkeyFaceMakePlanar",
@@ -166,14 +159,12 @@ def main():
         MeshTest("PlaneFillGrid", "testPlaneFillGrid",
                  "expectedPlaneFillGrid",
                  [OperatorSpecEditMode("fill_grid", {}, "EDGE", {1, 2, 3, 4, 5, 7, 9, 10, 11, 12, 13, 15})]),
-
         MeshTest("PlaneFillGridSimpleBlending",
                  "testPlaneFillGridSimpleBlending",
                  "expectedPlaneFillGridSimpleBlending",
                  [OperatorSpecEditMode("fill_grid", {"use_interp_simple": True}, "EDGE",
                                        {1, 2, 3, 4, 5, 7, 9, 10, 11, 12, 13, 15})]),
 
-        ### 31
         # fill holes
         MeshTest("SphereFillHoles", "testSphereFillHoles", "expectedSphereFillHoles",
                  [OperatorSpecEditMode("fill_holes", {"sides": 9}, "VERT", {i for i in range(481)})]),
@@ -184,7 +175,6 @@ def main():
                                                                              {5, 16, 17, 19, 20, 22, 23, 34, 47, 49, 50,
                                                                               52,
                                                                               59, 61, 62, 65, 83, 91, 95})]),
-
         MeshTest("CubeInsetEvenOffsetFalse",
                  "testCubeInsetEvenOffsetFalse", "expectedCubeInsetEvenOffsetFalse",
                  [OperatorSpecEditMode("inset", {"thickness": 0.2, "use_even_offset": False}, "VERT",



More information about the Bf-blender-cvs mailing list