Skip to content
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

Closed
serg06 opened this issue Nov 5, 2020 · 12 comments
Closed

Zen 3 CPUs #686

serg06 opened this issue Nov 5, 2020 · 12 comments
Assignees

Comments

@serg06
Copy link
Contributor

serg06 commented Nov 5, 2020

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.

@MarioAC2
Copy link

MarioAC2 commented Nov 5, 2020

That would be amazing, I got a 3090 thanks to this tool and now I ran out of luck again for the 5950x

@stevezub
Copy link

stevezub commented Nov 5, 2020

I'd love to see this, too. Waited in line at Micro Center for a while this morning for a 5900x, but no luck.

@The-Distal
Copy link

The-Distal commented Nov 5, 2020

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

	{
		brand: 'amd',
		cartUrl: 'https://www.amazon.com/gp/aws/cart/add.html?ASIN.1=B0815Y8J9N&Quantity.1=1',
		model: '5950x',
		series: '5950',
		url: 'https://www.amazon.com/dp/product/B0815Y8J9N'
	}

Newegg

	{
		brand: 'amd',
		cartUrl: 'https://secure.newegg.com/Shopping/AddtoCart.aspx?Submit=ADD&ItemList=N82E16819113663',
		itemNumber: '19-113-663',
		model: '5950x',
		series: '5950',
		url: 'https://www.newegg.com/amd-ryzen-9-5950x/p/N82E16819113663'
	}

@stevezub
Copy link

stevezub commented Nov 5, 2020

Thanks! Since I'm interested in the 3900x, I took a stab at modifying these for the 3900x:

Amazon

	{
		brand: 'amd',
		cartUrl: 'https://www.amazon.com/gp/aws/cart/add.html?ASIN.1=B08164VTWH&Quantity.1=1',
		model: '5900x',
		series: '5900',
		url: 'https://www.amazon.com/dp/product/B08164VTWH'
	}

Newegg

	{
		brand: 'amd',
		cartUrl: 'https://secure.newegg.com/Shopping/AddtoCart.aspx?Submit=ADD&ItemList=N82E16819113664',
		itemNumber: '19-113-664',
		model: '5900x',
		series: '5900',
		url: 'https://www.newegg.com/amd-ryzen-9-5900x/p/N82E16819113664'
	}

@The-Distal
Copy link

Thanks! Since I'm interested in the 3900x, I took a stab at modifying these for the 3900x:

Amazon

	{
		brand: 'amd',
		cartUrl: 'https://www.amazon.com/gp/aws/cart/add.html?ASIN.1=B08164VTWH&Quantity.1=1',
		model: '5900x',
		series: '5900',
		url: 'https://www.amazon.com/dp/product/B08164VTWH'
	}

Newegg

	{
		brand: 'amd',
		cartUrl: 'https://secure.newegg.com/Shopping/AddtoCart.aspx?Submit=ADD&ItemList=N82E16819113664',
		itemNumber: '19-113-664',
		model: '5900x',
		series: '5900',
		url: 'https://www.newegg.com/amd-ryzen-9-5900x/p/N82E16819113664'
	}

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.

@emike09
Copy link

emike09 commented Nov 5, 2020

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
16 brand: 'test:brand' | 'asus' | 'evga' | 'gainward' | 'gigabyte' | 'inno3d' | 'kfa2' | 'msi' | 'nvidia' | 'palit' | 'pny' | 'zotac';
~~~~~
The expected type comes from property 'brand' which is declared here on type 'Link'

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.

@stevezub
Copy link

stevezub commented Nov 5, 2020

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

@njgreb
Copy link
Contributor

njgreb commented Nov 5, 2020

That PR has been merged! Good luck on getting those sweet new cpu's ;)

@jef jef closed this as completed Nov 5, 2020
@emike09
Copy link

emike09 commented Nov 5, 2020

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?

@njgreb
Copy link
Contributor

njgreb commented Nov 5, 2020

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.

@njgreb
Copy link
Contributor

njgreb commented Nov 5, 2020

But to add them in your .env:
SHOW_ONLY_SERIES="ryzen5950,ryzen5900,ryzen5800,ryzen5600"

@njgreb
Copy link
Contributor

njgreb commented Nov 5, 2020

PR with them in the default: #696

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants