papampi / nvOC_miners

nvOC easy-to-use Linux Nvidia Mining OS Miners
8 stars 7 forks source link

Merge 3.0 miners to 3.1 #28

Closed papampi closed 5 years ago

LuKePicci commented 5 years ago

Happy new year to you too! If you capture the startup output of tempcontrol you should be able to see errors for each gpu that has failed in enabling manual fan control. The soft approach is to retry executing that command (line 158) for each gpu that failed in applying that setting by parsing the errors. The hard one is to keep setting FanControlState to 1 on each fan speed change. Btw this is yet another workaround for poor nvidia utility/driver stability.

Da: Payam Nabmailto:notifications@github.com Inviato: sabato 5 gennaio 2019 11:29 A: papampi/nvOC_minersmailto:nvOC_miners@noreply.github.com Cc: Subscribedmailto:subscribed@noreply.github.com Oggetto: [papampi/nvOC_miners] Merge 3.0 miners to 3.1 (#28)

You can view, comment on, or merge this pull request online at:

https://github.com/papampi/nvOC_miners/pull/28

Commit Summary

File Changes

Patch Links:

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/papampi/nvOC_miners/pull/28, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AIUXtmjt3kNq8QZPIAEPiRc8EOvwPtWVks5vAH6DgaJpZM4ZwZVP.

papampi commented 5 years ago

@LuKePicci It get me 5 minutes to figured out you replied to my email in this PR, was thinking whats merging miners 3.0 to 3.1 got to do with tempcontrol ...lol

Problem is I see no error in tempcontrol startup and all looks ok.

LuKePicci commented 5 years ago

I'm sorry, I mistapped the wrong reply button...

Yeah, because it is redirecting output to /dev/null at line 158.