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

llm plot visual improvements #1354

Closed
wants to merge 1 commit into from
Closed

Conversation

csauper
Copy link
Contributor

@csauper csauper commented Sep 19, 2024

Summary:

  1. size of seq plot adapts to data size
  2. axes background is white for visibility
  3. use red/white/blue scale for visibility + colorblind safety

Differential Revision: D63039687

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D63039687

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D63039687

csauper added a commit to csauper/captum that referenced this pull request Sep 19, 2024
Summary:
Pull Request resolved: pytorch#1354

1) size of seq plot adapts to data size
2) axes background is white for visibility
3) use red/white/blue scale  for visibility + colorblind safety

Reviewed By: jjuncho

Differential Revision: D63039687
Summary:
Pull Request resolved: pytorch#1354

1) size of seq plot adapts to data size
2) axes background is white for visibility
3) use red/white/blue scale  for visibility + colorblind safety

Reviewed By: jjuncho

Differential Revision: D63039687
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D63039687

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 49d8689.

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

Successfully merging this pull request may close these issues.

2 participants