-
Notifications
You must be signed in to change notification settings - Fork 30.2k
Update Support image on README.md #16615
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
Conversation
Updates the Support image linking to our EAP page (to give it a refresh + help avoid image fatigue). Slack thread checking in with #open-source-internal on this update (https://huggingface.slack.com/archives/C021H1P1HKR/p1648838903316709)
The documentation is not available anymore as the PR was closed or merged. |
whenever possible you should compress images using https://tinypng.com/ Here you'd be saving 83% for instance: Other than that, looks good to me! cc'ing @gary149 in case he has feedback on the design |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Compressed EAP/Support image via tinypng
Looks ok to me, but the logo is distorted and maybe the image should be vertically smaller to take less space on the readme? |
The logo shouldn't be distorted. Have tested this and the original image was verified/improved by our graphic designer, Bibi. @feconroses if you have any other thoughts? @gary149 could you share additional details/screenshots to help us address any distortions? |
Looks good to me! |
i think what @gary149 means is the logo is not the usual aspect ratio, it's "flatter". (let me invite Bibi to GitHub BTW) |
Hi! Bibi here. I'll have a look at the issue and resolve it |
Updated the image based on logo + size feedback. Big thanks to Bibi for making quick edits to this image.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok for me. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great to me, thank you @BritneyMuller!
you should tweet about it @BritneyMuller! |
What does this PR do?
Updates the Support image linking to our Expert Acceleration Program Page.
The Marketing + Monetization Team felt this image deserved a refresh to better highlight the incredible Machine Learning Experts at Hugging Face. In addition, we hope this helps drive additional EAP awareness among our brilliant community members in an effort to better support their ML work/roadmaps.