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

It seems that visualize_attention_scores can only visualize either mask-mod-only or score-mod-only #29

Closed
XinDongol opened this issue Aug 23, 2024 · 2 comments · Fixed by #86
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@XinDongol
Copy link

def visualize_attention_scores(

@drisspg
Copy link
Contributor

drisspg commented Aug 26, 2024

Do you have an example that needs both? It should deff be possible to extend it to visualize both

@drisspg drisspg added enhancement New feature or request good first issue Good for newcomers labels Aug 26, 2024
@XinDongol
Copy link
Author

XinDongol commented Aug 26, 2024

For example, if we want to apply Alibi positional embedding/soft-cap within the sliding window, a visualize_attention_scores() that supports both will be very useful.

@kashif kashif mentioned this issue Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants