Skip to content

Commit f4e90bd

Browse files
authored
Fix inappropriate code syntax that causes Python SyntaxWarning: "is not" with a literal (#246)
1 parent 93635f5 commit f4e90bd

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

pyrep/objects/octree.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -38,15 +38,15 @@ def insert_voxels(self, points: List[float], color: Optional[float] = None,
3838
if not isinstance(points, list):
3939
raise ValueError(
4040
'Octree.insert_voxels: points parameter is not a list.')
41-
if len(points) % 3 is not 0:
41+
if len(points) % 3 != 0:
4242
raise ValueError(
4343
'Octree.insert_voxels: points parameter length '
4444
'not a multiple of 3.')
4545
if color is not None:
4646
if not isinstance(color, list):
4747
raise ValueError(
4848
'Octree.insert_voxels: color parameter not a list.')
49-
elif len(color) is not 3:
49+
elif len(color) != 3:
5050
raise ValueError(
5151
'Octree.insert_voxels: color parameter not an RGB list.')
5252
sim.simInsertVoxelsIntoOctree(self._handle, options, points, color,None)
@@ -63,7 +63,7 @@ def remove_voxels(self, points : Optional[List[float]],
6363
if not isinstance(points, list):
6464
raise ValueError(
6565
'Octree.insert_voxels: points parameter is not a list.')
66-
if len(points) % 3 is not 0:
66+
if len(points) % 3 != 0:
6767
raise ValueError(
6868
'Octree.insert_voxels: points parameter length '
6969
'not a multiple of 3.')
@@ -88,7 +88,7 @@ def insert_object(self, obj: Object, color: Optional[float] = None,
8888
if not isinstance(color, list):
8989
raise ValueError(
9090
'Octree.insert_object: color parameter not a list.')
91-
elif len(color) is not 3:
91+
elif len(color) != 3:
9292
raise ValueError(
9393
'Octree.insert_object: color parameter not an RGB list.')
9494
sim.simInsertObjectIntoOctree(self._handle, obj.get_handle(), options,
@@ -109,7 +109,7 @@ def check_point_occupancy(self, points: List[float],
109109
if not isinstance(points, list):
110110
raise ValueError(
111111
'Octree.check_point_occupancy: points parameter is not a list.')
112-
if len(points) % 3 is not 0:
112+
if len(points) % 3 != 0:
113113
raise ValueError(
114114
'Octree._check_point_occupancy: points parameter length '
115115
'not a multiple of 3.')
@@ -120,4 +120,4 @@ def clear_voxels(self) -> None:
120120
"""
121121
sim.simRemoveVoxelsFromOctree(self._handle, 0, None)
122122

123-
object_type_to_class[ObjectType.OCTREE] = Octree
123+
object_type_to_class[ObjectType.OCTREE] = Octree

0 commit comments

Comments
 (0)