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.

Sign up for GitHub

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

Jump to bottom

PaddlePaddle Hackathon 52 提交 #1085

Merged
merged 25 commits into from
Oct 20, 2021
Merged

PaddlePaddle Hackathon 52 提交 #1085

merged 25 commits into from
Oct 20, 2021

Conversation

JunnYu
Copy link
Member

@JunnYu JunnYu commented Sep 23, 2021 •
edited
Loading

Task: #1078

  • 权重文件等百度云上传， 链接：https://pan.baidu.com/s/1WtTZaZLIMus843S2Eh8YAw 提取码：yv2b。
  • 删除modeling.py中没有使用过的包（强迫症）。
  • 添加compare.py比较转换后的模型预测结果。（该文件所处位置等审核通过后再修改）。
  • 添加convert.py转换脚本。（该文件所处位置等审核通过后再修改）。
  • 添加BertForMultipleChoice 和 BertForMaskedLM 这两个类。
  • 支持2d的attention mask。
  • 添加单元测试代码。TestBertForMultipleChoice和TestBertForMaskedLM。
  • windows条件下，paddle.int可能是int32，显示转换成int64。

@JunnYu JunnYu mentioned this pull request Sep 23, 2021
Closed
@yingyibiao yingyibiao self-assigned this Sep 24, 2021
@yingyibiao
Copy link
Contributor

新增模型权重通过添加community/junnyu/ 下的目录这种形式

@JunnYu
Copy link
Member Author

JunnYu commented Oct 13, 2021

新增模型权重通过添加community/junnyu/ 下的目录这种形式

已完成

yingyibiao
yingyibiao reviewed Oct 14, 2021
View reviewed changes
paddlenlp/transformers/bert/modeling.py Outdated Show resolved Hide resolved
paddlenlp/transformers/bert/modeling.py Outdated Show resolved Hide resolved
paddlenlp/transformers/bert/modeling.py Outdated Show resolved Hide resolved
paddlenlp/transformers/bert/tokenizer.py Outdated Show resolved Hide resolved
paddlenlp/transformers/bert/tokenizer.py Outdated Show resolved Hide resolved
paddlenlp/transformers/bert/modeling.py Outdated Show resolved Hide resolved
paddlenlp/transformers/bert/modeling.py Outdated Show resolved Hide resolved
paddlenlp/transformers/bert/modeling.py Outdated Show resolved Hide resolved
@JunnYu
Copy link
Member Author

JunnYu commented Oct 14, 2021

好的，明天修改完成

yingyibiao
yingyibiao reviewed Oct 18, 2021
View reviewed changes
community/junnyu/ckiplab-bert-base-chinese-ner/README.md Outdated Show resolved Hide resolved
community/junnyu/ckiplab-bert-base-chinese-ner/README.md Outdated Show resolved Hide resolved
community/junnyu/ckiplab-bert-base-chinese-pos/README.md Outdated Show resolved Hide resolved
community/junnyu/ckiplab-bert-base-chinese-pos/README.md Outdated Show resolved Hide resolved
community/junnyu/ckiplab-bert-base-chinese-ws/README.md Outdated Show resolved Hide resolved
paddlenlp/transformers/bert/modeling.py Outdated Show resolved Hide resolved
paddlenlp/transformers/bert/modeling.py Outdated Show resolved Hide resolved
paddlenlp/transformers/bert/modeling.py Outdated Show resolved Hide resolved
paddlenlp/transformers/bert/modeling.py Outdated Show resolved Hide resolved
paddlenlp/transformers/bert/modeling.py Show resolved Hide resolved
@yingyibiao
Copy link
Contributor

权重等文件已上传bos～

@JunnYu
Copy link
Member Author

JunnYu commented Oct 18, 2021

aistudio： https://aistudio.baidu.com/aistudio/projectdetail/2485265
image

This was referenced Oct 18, 2021
Merged
Merged
@yingyibiao
Copy link
Contributor

@joey12300 麻烦review一下 test_modeling.py 文件

yingyibiao
yingyibiao reviewed Oct 18, 2021
View reviewed changes
paddlenlp/transformers/bert/modeling.py Show resolved Hide resolved
community/junnyu/tbs17-MathBERT/README.md Outdated Show resolved Hide resolved
community/junnyu/ckiplab-bert-base-chinese-ws/README.md Outdated Show resolved Hide resolved
community/junnyu/ckiplab-bert-base-chinese-pos/README.md Outdated Show resolved Hide resolved
community/junnyu/ckiplab-bert-base-chinese-ner/README.md Outdated Show resolved Hide resolved
community/junnyu/bert_compare.py Show resolved Hide resolved
@yingyibiao
Copy link
Contributor

https://github.com/PaddlePaddle/PaddleNLP/blob/develop/docs/model_zoo/transformers.rst
这个文件也需要同步修改

yingyibiao
yingyibiao approved these changes Oct 20, 2021
View reviewed changes
Copy link
Contributor

@yingyibiao yingyibiao left a comment

Choose a reason for hiding this comment

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

LGTM

@yingyibiao yingyibiao merged commit c862e9c into PaddlePaddle:develop Oct 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yingyibiao yingyibiao yingyibiao approved these changes

Assignees

@yingyibiao yingyibiao

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@JunnYu @yingyibiao