-
Notifications
You must be signed in to change notification settings - Fork 302
Conversation
1bd2c8a
to
f402118
Compare
bceb057
to
6b48d9a
Compare
Wierd segfault during training...
other configs
|
|
Seemingly working with the current master EDIT:
EDIT: |
_canonical_scale = 224 | ||
_roi_size = 14 | ||
_roi_sample_ratio = 2 | ||
segm_size = _roi_size * 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.
Why is this member public?
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.
This is necessary for mask_head_loss_pre
.
To circumvent this problem, should we make mask_head_loss_pre
as a method of MaskHead
?
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 see. It is ok to make this member public.
|
||
:obj:`imgs`, ":math:`[(3, H, W)]`", :obj:`float32`, \ | ||
"RGB, :math:`[0, 255]`" | ||
:obj:`rois`, ":math:`[(R', 4)]`", :obj:`float32`, \ |
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.
Can we add a line between inputs part and outputs part? (or just make two tables)
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
# pad with repeated border values), we manually zero-pad the masks by 1 | ||
# pixel prior to resizing back to the original image resolution. | ||
# This prevents "top hat" artifacts. We therefore need to expand | ||
# the reference boxes by an appropriate factor. |
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 have two similar blocks. Can we merge them?
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
|
pfnCI, test this please |
pfnCI, test this please |
pfnCI, test this please |
pfnCI, test this please |
Merge after #778