-
Notifications
You must be signed in to change notification settings - Fork 306
Conversation
.. csv-table:: | ||
:header: name, shape, dtype, format | ||
|
||
:obj:`point`, ":math:`[(K, 2)]` or :math:`(R, K, 2)`", \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In Args
section, ndarray
comes first.
point (~numpy.ndarray or list of arrays)
I prefer to use the same order here.
:obj:`point`, ":math:`(R, K, 2)` or :math:`[(K, 2)]`"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you
np.testing.assert_equal(out, point_expected) | ||
|
||
def test_flip_point_list(self): | ||
point = [np.random.uniform( | ||
low=0., high=32., size=(12, 2))] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about using point
with more than one samples?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK
def test_flip_point_list(self): | ||
point = [ | ||
np.random.uniform(low=0., high=32., size=(12, 2)), | ||
np.random.uniform(low=0., high=32., size=(12, 2)), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I prefer to use two arrays of different shapes (e.g. (12, 2)
and (10, 2)
).
def test_resize_point_list(self): | ||
point = [ | ||
np.random.uniform(low=0., high=32., size=(12, 2)), | ||
np.random.uniform(low=0., high=32., size=(12, 2)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ditto.
def test_translate_point_list(self): | ||
point = [ | ||
np.random.uniform(low=0., high=32., size=(10, 2)), | ||
np.random.uniform(low=0., high=32., size=(10, 2)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ditto.
Assume
point
as[(K, 2)]
instead of(K, 2)
.