-
Notifications
You must be signed in to change notification settings - Fork 306
Conversation
|
||
:obj:`img`, ":math:`(3, H, W)`", :obj:`float32`, \ | ||
"RGB, :math:`[0, 255]`" | ||
:obj:`point`, ":math:`[(K, 2)]`", :obj:`float32`, ":math:`(y, x)`" |
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.
If we assumes all instances have the same K
, it should be (N, K, 2)
(N
is the number of instances).
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 #808, R
is used for the number of instances. It's better to make them consistent.
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.
We don't assume that all instances have the same K
for this function.
That is why I left the documentation to accept lists.
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.
So, these two lines should be like (~numpy.ndarray or list of arrays)
.
chainercv/chainercv/visualizations/vis_point.py
Lines 24 to 25 in 07f136e
point (~numpy.ndarray): See the table below. | |
visible (~numpy.ndarray): See the table below. |
fixed |
|
||
:obj:`img`, ":math:`(3, H, W)`", :obj:`float32`, \ | ||
"RGB, :math:`[0, 255]`" | ||
: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.
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.
Ah. Thanks
"RGB, :math:`[0, 255]`" | ||
:obj:`point`, ":math:`(R, K, 2)` or :math:`[(K, 2)]`", \ | ||
:obj:`float32`, ":math:`(y, x)`" | ||
:obj:`visible`, ":math:`[(K,)]` or :math:`(R, K)`", :obj:`bool`, -- |
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.
":math:`(R, K)` or :math:`[(K,)]`"
Assume
point
as[(K, 2)]
instead of(K, 2)
.