-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Zen 3 CPUs #686
Comments
That would be amazing, I got a 3090 thanks to this tool and now I ran out of luck again for the 5950x |
I'd love to see this, too. Waited in line at Micro Center for a while this morning for a 5900x, but no luck. |
This is not mine https://github.com/njgreb/nvidia-snatcher/tree/amd5950 This is mine The branch above only includes support for the 5950x at AMD and Bestbuy in the US. Here's the code to edit the files in \nvidia-snatcher\src\store\model for 5950x support at Amazon and Newegg. Amazon
Newegg
|
Thanks! Since I'm interested in the 3900x, I took a stab at modifying these for the 3900x: Amazon
Newegg
|
Let me know if it works. I haven't tried new models. I'm not much of a programmer; I just recognized enough that I knew that edit would work. |
I get an error when I add those entries. src/store/model/amazon.ts:26:4 - error TS2322: Type '"amd"' is not assignable to type '"nvidia" | "test:brand" | "asus" | "evga" | "gainward" | "gigabyte" | "inno3d" | "kfa2" | "msi" | "palit" | "pny" | "zotac"'. 26 brand: 'amd', src/store/model/store.ts:16:2 src/store/model/amazon.ts:29:4 - error TS2322: Type '"5950"' is not assignable to type 'Series'. 29 series: '5950', I assume I need to edit another files so that AMD is recognized? EDIT: You also have to edit store.ts and add the series and brand. Make sure they match to what you've put in the other files. |
You would need to pull down the rest of the changes in https://github.com/njgreb/nvidia-snatcher/tree/amd5950. He's adding the rest of the Ryzen 5000 series as well |
That PR has been merged! Good luck on getting those sweet new cpu's ;) |
I pulled the latest version of jef/streetmerchant, added amd to stores in .env, but I'm not seeing and AMD chips in the display output. Just nvidia cards. I don't have and limits on showing any brands, models, or series. Is there something else that needs to be done to look for AMD chips? |
The default set of devices is still "['3070', '3080', '3090']". I didn't add these to the set since it felt a bit odd with the "nvidia-snatcher" name. With the new name, I am less against that and could do make a change to include them all. |
But to add them in your .env: |
PR with them in the default: #696 |
Description
Would it be possible to add the Zen 3 CPUs to this tool? I know they're technically not Nvidia products, but this tool is amazing, and I'd love to be able to use it for Zen 3.
Possible solution
We currently have 3 series options, 3070 3080 3090. We could add 4 more, 5600x 5800x 5900x 5950x, to cover the Zen 3 CPUs. I think it should be really easy to do.
The text was updated successfully, but these errors were encountered: