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

DeepBBT : the outputs of modified Roberta don't have hidden states #14

Closed
Desein-Yang opened this issue Aug 17, 2022 · 2 comments
Closed

Comments

@Desein-Yang
Copy link

Desein-Yang commented Aug 17, 2022

Hello, there is a bug in RobertaL Model ouput when I run deepbbt because it do not have hidden state even the config.output_hidden_states = True (Line439).

Then I found the Line 975 of deep_modeling_roberta.py only output a dict with logit rather than the original MskedLMOutput, which leads to the above bug.

When I try to directly replace the logit of MaskedLMOutput to fix it, it suggest the inner model also do not output hidden state. I would appreciate if you could help to fix that !

@txsun1997
Copy link
Owner

Thanks for your issue! I've updated the deep_modeling_roberta.py, please try again :)

@Desein-Yang
Copy link
Author

Thank you very much :) It works well for me.

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

No branches or pull requests

2 participants