-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[Fnet P0]add PretrainedConfig and unit test #5780
Conversation
Thanks for your contribution! |
感谢贡献!我们目前ci有些小问题,正在修 |
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.
麻烦merge一下develop, ci应该ok了
Codecov Report
@@ Coverage Diff @@
## develop #5780 +/- ##
===========================================
+ Coverage 61.44% 61.84% +0.39%
===========================================
Files 489 490 +1
Lines 69028 68984 -44
===========================================
+ Hits 42417 42664 +247
+ Misses 26611 26320 -291
|
似乎还是有些问题 |
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.
多谢贡献~ 几个小问题
from .configuration import ( | ||
FNET_PRETRAINED_INIT_CONFIGURATION, | ||
FNET_PRETRAINED_RESOURCE_FILES_MAP, | ||
FnetConfig, |
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.
FnetConfig, | |
FNetConfig, |
} | ||
|
||
|
||
class FnetConfig(PretrainedConfig): |
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.
class FnetConfig(PretrainedConfig): | |
class FNetConfig(PretrainedConfig): |
@@ -1046,7 +922,8 @@ def forward( | |||
) | |||
sequence_output = outputs[0] if not return_dict else outputs["last_hidden_state"] | |||
logits = self.qa_outputs(sequence_output) | |||
start_logits, end_logits = paddle.split(logits, num_or_sections=1, axis=-1) | |||
print(logits.shape) |
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.
多余的print麻烦删掉~
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.
lgtm
谢谢您这段时间的帮助与指导
zhou
***@***.***
…------------------ 原始邮件 ------------------
发件人: "Sijun ***@***.***>;
发送时间: 2023年5月5日(星期五) 上午10:57
收件人: ***@***.***>;
抄送: ***@***.***>; ***@***.***>;
主题: Re: [PaddlePaddle/PaddleNLP] [Fnet P0]add PretrainedConfig and unit test (PR #5780)
@sijunhe approved this pull request.
lgtm
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
PR types
PR changes
Description