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

Add additional documentation for the use of custom images on the oled. #195

Merged
merged 6 commits into from
Nov 15, 2022

Conversation

redoxcode
Copy link
Contributor

@redoxcode redoxcode commented Nov 14, 2022

This PR adds additional documentation for the use of custom images on the oled display and also addresses potential screen burn in as mentioned here: #83

Added section about custom images
Added extra tips for custom images
Added information about Screen burn in
Copy link
Collaborator

@mjaskula mjaskula left a comment

Choose a reason for hiding this comment

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

Hello, thanks for your PR. It's always great to have some documentation updates. If you haven't yet, pleas read the contributing guide so that you know what to expect during your PR.

This one looks pretty good to me. I highlighted a couple typos to be corrected, and opened a discussion on the image used in the example. Once we have these resolved we can get this merged. Thanks again for your contribution. I hope that there will be more.

software/oled_tips.md Outdated Show resolved Hide resolved
software/oled_tips.md Outdated Show resolved Hide resolved
software/oled_tips.md Outdated Show resolved Hide resolved
Changed image bytearray and fixed typos
fixed another typo
fixed another typo
Copy link
Collaborator

@mjaskula mjaskula left a comment

Choose a reason for hiding this comment

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

Looks good to me now. Thanks!

@mjaskula mjaskula merged commit e9fc9b7 into Allen-Synthesis:main Nov 15, 2022
@roryjamesallen
Copy link
Collaborator

LGTM, thanks for adding to the docs!

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.

None yet

3 participants