Fix article generator producing 1xINTEL in GPU segment
MB_ lots (e.g. MB_INTEL_..._GPU8) are incorrectly categorized as GPU in the pricelist. Two fixes: - Skip MB_ lots in buildGPUSegment regardless of pricelist category - Add INTEL to vendor token skip list in parseGPUModel (was missing, unlike AMD/NV/NVIDIA which were already skipped) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -195,6 +195,9 @@ func buildGPUSegment(items []models.ConfigItem, cats map[string]string) string {
|
|||||||
if !ok || group != GroupGPU {
|
if !ok || group != GroupGPU {
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
|
if strings.HasPrefix(strings.ToUpper(it.LotName), "MB_") {
|
||||||
|
continue
|
||||||
|
}
|
||||||
model := parseGPUModel(it.LotName)
|
model := parseGPUModel(it.LotName)
|
||||||
if model == "" {
|
if model == "" {
|
||||||
model = "UNK"
|
model = "UNK"
|
||||||
@@ -332,7 +335,7 @@ func parseGPUModel(lotName string) string {
|
|||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
switch p {
|
switch p {
|
||||||
case "NV", "NVIDIA", "AMD", "RADEON", "PCIE", "PCI", "SXM", "SXMX":
|
case "NV", "NVIDIA", "INTEL", "AMD", "RADEON", "PCIE", "PCI", "SXM", "SXMX":
|
||||||
continue
|
continue
|
||||||
default:
|
default:
|
||||||
if strings.Contains(p, "GB") {
|
if strings.Contains(p, "GB") {
|
||||||
|
|||||||
Reference in New Issue
Block a user