Easy-Transformer icon indicating copy to clipboard operation
Easy-Transformer copied to clipboard

removed einsum causing error when use_atten_result is enabled

Open oliveradk opened this issue 1 year ago • 1 comments

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
Screenshot 2024-07-08 at 12 59 00 PM Screenshot 2024-07-08 at 12 59 40 PM

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

oliveradk avatar Jul 08 '24 17:07 oliveradk

@bryce13950 should I close this, or should we merge the test?

oliveradk avatar Jul 19 '24 15:07 oliveradk

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.

bryce13950 avatar Aug 15 '24 23:08 bryce13950