Skip to content

Fix dygraph show style#18297

Merged
junjun315 merged 17 commits intoPaddlePaddle:developfrom
junjun315:fix-show-style
Jun 27, 2019
Merged

Fix dygraph show style#18297
junjun315 merged 17 commits intoPaddlePaddle:developfrom
junjun315:fix-show-style

Conversation

@junjun315
Copy link
Contributor

Fix dygraph show style for FluidDoc

@junjun315 junjun315 requested review from JiabinYang and phlrain and removed request for JiabinYang June 24, 2019 12:14
@junjun315 junjun315 requested review from chengduoZH, liuyuuan and xsrobin and removed request for liuyuuan June 24, 2019 12:24
chengduoZH
chengduoZH previously approved these changes Jun 24, 2019
@junjun315 junjun315 closed this Jun 24, 2019
@junjun315 junjun315 reopened this Jun 24, 2019
xsrobin
xsrobin previously approved these changes Jun 24, 2019
@junjun315 junjun315 dismissed stale reviews from xsrobin and chengduoZH via ea9030b June 24, 2019 16:06
xsrobin
xsrobin previously approved these changes Jun 24, 2019
luotao1
luotao1 previously approved these changes Jun 25, 2019
ceil_mode (bool): ${ceil_mode_comment}
name (str|None): A name for this layer(optional). If set None, the
layer will be named automatically.
global_pooling (bool): (bool, default false) Whether to use the global pooling. If global_pooling = true,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

多了bool

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

layer will be named automatically.
global_pooling (bool): (bool, default false) Whether to use the global pooling. If global_pooling = true,
kernel size and paddings will be ignored
use_cudnn (bool): (bool, default True) Onlyceil_mode (bool) - (bool, default false) Whether to use the ceil
Copy link
Contributor

@xsrobin xsrobin Jun 25, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

多了bool,ceil_mode另起一行

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

global_pooling (bool): (bool, default false) Whether to use the global pooling. If global_pooling = true,
kernel size and paddings will be ignored
use_cudnn (bool): (bool, default True) Onlyceil_mode (bool) - (bool, default false) Whether to use the ceil
function to calculate output height and width. False is the default.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

删除False is the default.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

Copy link
Contributor

@xsrobin xsrobin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

as comments

@junjun315 junjun315 requested a review from xsrobin June 26, 2019 05:49
xsrobin
xsrobin previously approved these changes Jun 26, 2019
@junjun315 junjun315 requested review from chengduoZH and xsrobin June 27, 2019 05:13
@junjun315 junjun315 merged commit fd6631e into PaddlePaddle:develop Jun 27, 2019
@junjun315 junjun315 deleted the fix-show-style branch June 27, 2019 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants