yekyau / nfancurve-amd

a fork of nan0s7's script adapted for amdgpu users
GNU General Public License v3.0
1 stars 0 forks source link

Not an actual issue #1

Open nan0s7 opened 7 years ago

nan0s7 commented 7 years ago

Hey, just thought I'd pop over and say holy crap! This is so cool! :D I'm so happy you're doing this, and if you need any help understanding my current/future code feel free to contact me (I always use nan0s7 on the socials).

I'd also like to give you a heads up that I'm working on multi-GPU support in the next update, and I've also optimised the method the script uses to calculate the new fan-speed. Still ironing out some weird issues with the multiple graphics card support but we're pretty close.

But yeah, good luck with what you have planned for this. I look forward to seeing what you do in the future!

yekyau commented 7 years ago

Hey, thanks for making your script in the first place, and making it so easy to work with! All I really ended up having to do was swap out nvidia-settings with echo x > pwm1 in a couple of functions

I don't think amdgpu supports crossfire so I think I can ignore multi-gpu, but who knows what crazy setups people have. In theory they would just need to run 2 copies of the script with a different gpuid in each.

nan0s7 commented 7 years ago

Well I'm glad that it was easy to adjust for AMD! :D

Yeah I tried that and it works fine on the NVIDIA side, but I saw it as a chance to make things a bit better because it'd force me to make some optimisations and such. Plus it was something I had on my list of stuff I wanted to add.

It shouldn't be too difficult to merge your updates with the new version when it's ready anyway, so nothing to worry about! :)