Easy-Transformer
Easy-Transformer copied to clipboard
removed einsum causing error when use_atten_result is enabled
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes #659
Type of change
Please delete options that are not relevant.
- [X] Bug fix (non-breaking change which fixes an issue)
Screenshots
Please attach before and after screenshots of the change if applicable.
| Before | After |
|---|---|
Checklist:
- [X] I have commented my code, particularly in hard-to-understand areas
- [] I have made corresponding changes to the documentation
- [X] My changes generate no new warnings
- [X] I have added tests that prove my fix is effective or that my feature works
- [X] New and existing unit tests pass locally with my changes
- [X] I have not rewritten tests relating to key interfaces which would affect backward compatibility
@bryce13950 should I close this, or should we merge the test?
Sorry @oliveradk! The last month has been pretty busy, and this one slipped through the cracks. Looks perfect, and it will be in the next release.