lmarzen / esp32-weather-epd

A low-power E-Paper weather display powered by an ESP32 microcontroller. Utilizes the OpenWeatherMap API.
GNU General Public License v3.0
2.33k stars 179 forks source link

Change PIN_EPD_CS number in config.cpp #77

Closed octopuss closed 4 months ago

octopuss commented 5 months ago

Please could you change default config.cpp https://github.com/lmarzen/esp32-weather-epd/blob/ebfe399e4b00c0c02df7ed245b46f267b9326e57/platformio/src/config.cpp#L26 so it is not set to LED_BUILTIN on FireBeetle 2 ESP32-E (or write some comment there). It is also in the wiring diagrams. It took me too much time to figure out why my display shows random noise.

Thank you!

lmarzen commented 5 months ago

Wow I don't know how that hasn't been an issue until now. Great find! I will update this once I get a chance.

octopuss commented 5 months ago

My guess is that standard 7.3 inch waveshare displays are somehow ok with that( i have one project done almost a year ago) Now i am creating 2 inch (400x168 waveshare) reduced version for my desktop. Anyway cool roject. I've also connected it to Home assistant so it displays my outdoor temperature sensor values

On Fri, Jan 26, 2024, 22:27 Luke Marzen @.***> wrote:

Wow I don't know how that hasn't been an issue until now. Great find! I will update this once I get a chance.

— Reply to this email directly, view it on GitHub https://github.com/lmarzen/esp32-weather-epd/issues/77#issuecomment-1912718561, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAUSRXMYIHGQD362NXHBVIDYQQNU3AVCNFSM6AAAAABCMXYWN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJSG4YTQNJWGE . You are receiving this because you authored the thread.Message ID: @.***>

lmarzen commented 4 months ago

Thanks for the troubleshooting and precise identification of the problem. I have fixed the issue.