Skip to content

data(laptop): link cpu/gpu to the component catalog (1287 CPU + 509 GPU) - #104

Merged
Seungpyo1007 merged 2 commits into
mainfrom
data/import-staging
Jul 8, 2026
Merged

Seungpyo1007 merged 2 commits into
mainfrom
data/import-staging

Conversation

@Seungpyo1007

Copy link
Copy Markdown
Member

What & why

Quality pass on the laptop category: resolve each laptop's cpu_name/gpu_name string to an existing CPU/GPU record and set the optional cpu/gpu foreign keys. Laptop detail pages now link to the component catalog (e.g. a laptop → its Core i7-6700HQ and RTX 1060).

  • CPU links: 1,287 · GPU links (dedicated only): 509 · across 1,467 of 1,935 laptops.
  • Matching is exact on the model code (7200U, 13420H, RTX 3050, GTX 1650) with a family cross-check (i5/i7/ryzen-5) — no fuzzy guessing. Integrated GPUs (Intel HD/UHD/Iris) and chips not in the catalog stay name-only. Raw cpu_name/gpu_name are preserved.
  • No new records; 1,467 laptops modified + their dump details refreshed.

Commits (each Refs #1): the FK-linking data commit + a surgical laptop-dump refresh.

Closes #1

Checklist

  • python -m app.validate — ✅ passed (all cpu/gpu FKs resolve to real records)
  • git diff --check — ✅ clean
  • Only laptops modified; dump limited to the 1,467 affected detail files

Resolve laptop cpu_name/gpu_name strings to existing CPU/GPU records
and set the optional `cpu`/`gpu` FKs (1287 CPU + 509 GPU links across
1467 laptops). Matching is exact on the model code (e.g. 7200U, RTX
3050) with a family cross-check; integrated GPUs and unmatched chips
stay name-only. Raw cpu_name/gpu_name are kept.

Refs #1
Update the 1467 laptop detail files that gained a cpu/gpu embed after
linking cpu_name/gpu_name to the component catalog. Regenerated via
app.dump @ d1c3b0b; only the affected laptop details are committed.

Refs #1
@Seungpyo1007

Copy link
Copy Markdown
Member Author

🤖 Review — PASS

Check Result
python -m app.validate ✅ passed (all 1,796 new cpu/gpu FKs resolve to real records)
git diff --check ✅ clean

Change: 1,287 CPU + 509 GPU foreign keys added across 1,467 laptops (of 1,935). Exact model-code match (7200U, RTX 3050) + family cross-check; integrated GPUs and catalog-missing chips stay name-only. No new records. Dump limited to the 1,467 affected laptop detail files.

@Seungpyo1007
Seungpyo1007 merged commit 8bba834 into main Jul 8, 2026
5 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in TechAPI-Project Jul 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

data Dataset changes enhancement New feature or request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

TechAPI dataset roadmap and status: all categories (1989-2026)

1 participant