Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

映射文档 No.51 - 60 #5919

Merged
merged 3 commits into from
Jun 27, 2023
Merged

映射文档 No.51 - 60 #5919

merged 3 commits into from
Jun 27, 2023

Conversation

co63oc
Copy link
Contributor

@co63oc co63oc commented Jun 3, 2023

#5691

映射文档 No.51, 52, 53, 55, 56, 58, 60

paddle.nn.functional.tanh paddle无文档
torch.Tensor.eig pytorch无文档
paddle.Tensor.nan_to_num paddle无文档
torch.Tensor.qr paddle无文档

部分API未添加为已有文档

@paddle-bot
Copy link

paddle-bot bot commented Jun 3, 2023

感谢你贡献飞桨文档,文档预览构建中,Docs-New 跑完后即可预览,预览链接:http://preview-pr-5919.paddle-docs-preview.paddlepaddle.org.cn/documentation/docs/zh/api/index_cn.html
预览工具的更多说明,请参考:飞桨文档预览工具

Copy link
Collaborator

@Skylark-hjyp Skylark-hjyp left a comment

Choose a reason for hiding this comment

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

全局将paddle暂无转写方式替换为paddle无此参数,暂无转写方式 吧,和其他文件保持一致

paddle.nn.RNNCellBase(name_scope=None, dtype='float32')
```

需要调用 get_initial_states(batch_ref,shape=None,dtype=None,init_value=0.,batch_dim_idx=0) 生成初始状态。
Copy link
Collaborator

Choose a reason for hiding this comment

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

把这个函数里的加上,下面的参数有能对上的吗

Copy link
Contributor Author

Choose a reason for hiding this comment

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

RNNCellBase 不能匹配,修改为匹配paddle.nn.SimpleRNN

### [paddle.DataParallel](https://www.paddlepaddle.org.cn/documentation/docs/zh/api/paddle/DataParallel_cn.html)

```python
paddle.DataParallel(layers, strategy=None, comm_buffer_size=25, last_comm_buffer_size=1, find_unused_parameters=False)
Copy link
Collaborator

Choose a reason for hiding this comment

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

这两个参数差异比较大,是还需要其他的一些操作吗,比如rnnbase那样

Copy link
Contributor Author

Choose a reason for hiding this comment

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

文档中没有说明其他操作

@zhwesky2010
Copy link
Collaborator

你好,请修复一下冲突

@co63oc
Copy link
Contributor Author

co63oc commented Jun 20, 2023

已修改冲突

@luotao1 luotao1 self-assigned this Jun 26, 2023
@zhwesky2010
Copy link
Collaborator

另外,请处理一下冲突

@co63oc
Copy link
Contributor Author

co63oc commented Jun 27, 2023

另外,请处理一下冲突

已修改

torch.nn.RNNBase(mode, input_size, hidden_size, num_layers=1, bias=True, batch_first=False, dropout=0.0, bidirectional=False, proj_size=0, device=None, dtype=None)
```

### [paddle.nn.SimpleRNN](https://www.paddlepaddle.org.cn/documentation/docs/zh/api/paddle/nn/SimpleRNN_cn.html)
Copy link
Collaborator

Choose a reason for hiding this comment

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

刚看了下,这个对应paddle.nn.RNNBase更好,SimpleRNN只是其一种mode

Copy link
Contributor Author

@co63oc co63oc Jun 27, 2023

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

paddle.nn.layer.rnn.RNNBase 没有api文档,删除差异文档

Copy link
Collaborator

@zhwesky2010 zhwesky2010 Jun 27, 2023

Choose a reason for hiding this comment

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

为啥没有paddle.nn.RNNBase呢,源码里是有paddle.nn.RNNBase的,以代码为主

Copy link
Collaborator

@zhwesky2010 zhwesky2010 left a comment

Choose a reason for hiding this comment

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

LGTM

@zhwesky2010 zhwesky2010 merged commit f0d96a4 into PaddlePaddle:develop Jun 27, 2023
@co63oc co63oc deleted the doc0602 branch July 25, 2023 05:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants