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

Fix missing argument attn_dropout in imputation Transformer #263

Merged

WenjieDu merged 1 commit into main from dev

Dec 10, 2023

Owner

WenjieDu commented Dec 10, 2023

What does this PR do?

Add back the missing argument attn_dropout.

Before submitting

I have commented my code, particularly in hard-to-understand areas.
I have written necessary tests and already run them locally.


fix: missing argument attn_dropout;

b3ee912

Collaborator

coveralls commented Dec 10, 2023 •

edited

Loading

Pull Request Test Coverage Report for Build 7157206024

2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 83.022%

Files with Coverage Reduction	New Missed Lines	%
pypots/modules/transformer/attention.py	1	98.08%

Totals
Change from base Build 7144339438:	0.0%
Covered Lines:	4802
Relevant Lines:	5784

💛 - Coveralls

WenjieDu merged commit 2a5ded1 into main

13 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment