NerdMiner_v2 icon indicating copy to clipboard operation
NerdMiner_v2 copied to clipboard

t-hmi model improvements and bugfixes

Open cosmicpsyop opened this issue 1 year ago • 3 comments

1/ Enable battery option

2/ Improve pool data api URL discovery based on Settings.PoolAddress and/or Settings.PoolPort it is possible that this is applicable to ESP32_2432S028R || ESP32_2432S028_2USB || ESP32_2432S024 || NERDMINER_T_HMI

cosmicpsyop avatar Apr 14 '24 04:04 cosmicpsyop

I've seen your changes, should not be any problem. Just one question. Why you changed so many pins on this board since previous release? I can't test it because I don't have this board, so I'll assume you are right on your changes. Can you provide some photos or graphic results?

BitMaker-hub avatar Apr 29 '24 20:04 BitMaker-hub

There are no UI changes affected by this PR although one could contend that there should be. I realized when I was testing the different pools that support that API the title is part of the graphic and remains Public-Pool.io. It probably should be dynamically replaced with the pool name entered. I did not have time to reach out to a ESP32_2432S028 maintainer to if could get a hand with coverage.

The pins have always been confusing for me in terms of the naming convention when reading from the lilygo repo. I was trying to align to nerdminer standards while trying to sort out the sdcard functionality which is critical as I understand for the SDCard class. In the end, I could not get cards recognized with the 1-Bit configuration and filed a ticket for help https://github.com/Xinyuan-LilyGO/T-HMI/issues/27

I left a bit of a mess in that file trying different things. Same could be said for the touch screen and the third button.

cosmicpsyop avatar Apr 29 '24 23:04 cosmicpsyop

@BitMaker-hub cleaned up lilygoT_HMI.h, leaving SDMMC_1BIT and touchscreen disabled for now.

cosmicpsyop avatar May 07 '24 18:05 cosmicpsyop