Summary:

The chipset is no longer a one-to-one mapping with the vendor ids. I updated @Raymo111's commit to move the chipset away from the product id decorator and instead something that's assigned within each class' __init__ method. Semantically, this makes the most sense to me that a device's class should be the one to figure out its own chipset. This being in the decorator made sense but it mixed with the product id logic, which isn't nice; I would instead prefer it to be a separate decorator that handles chipsets, but parsing a mapping in __init__ of the respective class makes more sense to me than an external method.

Sorry for bikeshedding over such a simple change; I sometimes* obsess over semantics :)

Note to @Raymo111, don't merge this. If you approve, just approve and I'll rebase to pop the older commit, add you as a co-author on mine and then merge.

Note to @Raymo111, don't merge this. If you approve, just approve and I'll rebase to pop the older commit, add you as a co-author on mine and then merge.

Can't we just squash merge

Note to @Raymo111, don't merge this. If you approve, just approve and I'll rebase to pop the older commit, add you as a co-author on mine and then merge.

Can't we just squash merge

Yeah, the reason I said that initially was because of this line: