Skip to content

Replace contributors table with DeepSpeed-style circular avatars#89

Merged
anxiangsir merged 2 commits intomainfrom
copilot/update-avatar-to-circular
Feb 7, 2026
Merged

Replace contributors table with DeepSpeed-style circular avatars#89
anxiangsir merged 2 commits intomainfrom
copilot/update-avatar-to-circular

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 7, 2026

Replace the manual HTML <table> contributors section with auto-generated circular avatars via contrib.rocks, matching the DeepSpeed README style.

  • Removes manually maintained 7-column table with rectangular avatars + usernames
  • Uses a single <img> from contrib.rocks that auto-generates circular avatar-only layout from repo contributors
<a href="https://github.com/EvolvingLMMs-Lab/OneVision-Encoder/graphs/contributors">
  <img src="https://contrib.rocks/image?repo=EvolvingLMMs-Lab/OneVision-Encoder" width="800px"/>
</a>
Original prompt

你可以改成deepspeed那种形式,只有头像的那种,圆形的
,谢谢


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…g contrib.rocks

Co-authored-by: anxiangsir <31175974+anxiangsir@users.noreply.github.com>
Copilot AI changed the title [WIP] Update user avatars to circular format Replace contributors table with DeepSpeed-style circular avatars Feb 7, 2026
Copilot AI requested a review from anxiangsir February 7, 2026 15:07
@anxiangsir anxiangsir marked this pull request as ready for review February 7, 2026 15:16
Copilot AI review requested due to automatic review settings February 7, 2026 15:16
@anxiangsir anxiangsir merged commit f58cc6e into main Feb 7, 2026
@anxiangsir anxiangsir deleted the copilot/update-avatar-to-circular branch February 7, 2026 15:17
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the README’s Contributors section to use a DeepSpeed-style, auto-generated circular avatar strip via contrib.rocks, removing the manually maintained HTML table.

Changes:

  • Replaced the manual 7-column contributors <table> with a contrib.rocks-generated contributors image link.
  • Simplified the Contributors section markup to a single link + image.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md
</tr>
</table>
<a href="https://github.com/EvolvingLMMs-Lab/OneVision-Encoder/graphs/contributors">
<img src="https://contrib.rocks/image?repo=EvolvingLMMs-Lab/OneVision-Encoder" width="800px"/>
Copy link

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The new contributors is missing an alt attribute, while other README images consistently include alt text. Consider adding a descriptive alt (e.g., "Contributors") to improve accessibility/screen reader support.

Suggested change
<img src="https://contrib.rocks/image?repo=EvolvingLMMs-Lab/OneVision-Encoder" width="800px"/>
<img src="https://contrib.rocks/image?repo=EvolvingLMMs-Lab/OneVision-Encoder" alt="Contributors" width="800px"/>

Copilot uses AI. Check for mistakes.
Comment thread README.md
</tr>
</table>
<a href="https://github.com/EvolvingLMMs-Lab/OneVision-Encoder/graphs/contributors">
<img src="https://contrib.rocks/image?repo=EvolvingLMMs-Lab/OneVision-Encoder" width="800px"/>
Copy link

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

width="800px" is inconsistent with the rest of the README (which uses numeric pixel widths like width="800") and HTML width attributes are typically numeric. Consider changing to width="800" and/or adding style="max-width: 100%;" so the image scales better on narrow screens.

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants