feat(data): add 7 GPUs from Wikipedia list pages - #327
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What & why
A live reconciliation of the four Wikipedia GPU pages against current TechAPI records found 7 genuinely absent, complete models, all Nvidia. The 1,581 missing entries in TechEngine #19 are overwhelmingly non-actionable (1,574 / 99.6%): existing names with different punctuation, branding, memory/interface/mobile qualifiers; ATI records compared only against AMD; incomplete models; integrated graphics; and non-model cells caused by reading the first physical cell without expanding rowspans.
gpu/atirecords)Expand rowspan/colspan tables, select actual model columns (including Intel's separate branding/model columns), normalize both names and slugs ignoring punctuation, and inspect substring matches in both directions. A shared prefix alone is not a duplicate: MX 440 SE/8x, FX 5700 LE, plain FX 5900, and plain/XT 6800 have different specifications from the existing related cards.
Add GeForce4 MX 420, MX 440 SE, MX 440 8x, GeForce FX 5700 LE, FX 5900, GeForce 6800, and 6800 XT. These names cover 17 complete table configuration rows; add one explicitly sourced configuration per model, with no cross-row mixing. The historical single core clock is stored as both base and boost, following the dataset's pre-boost convention. Every addition is
verified: false; no benchmark values are inferred.Examples checked by hand: Radeon X1550 matches existing AGP/PCI entries; Radeon HD 2350 matches HD 2350 PRO; GeForce FX 5700 E.S. matches Engineering Sample; GTX 1060/1650 Ti and RTX 2050 mobile rows match existing mobile records; Quadro T500 Mobile matches T500 Mobile; RTX PRO 6000 Blackwell Max-Q Workstation Edition matches the shorter Max-Q name; Intel A770 16GB matches Arc A770's 16GB record. Arc Pro B70 is source-only but its single combined clock cell does not clearly establish both required base and boost clocks; no record is added by guessing. Older mobile cards often lack TDP, and integrated Intel graphics remain outside the discrete GPU scope.
The final commit regenerates GPU detail/score files, the GPU index, and the GPU manifest entry with the normal TechEngine seed/API dump code and a fresh isolated database. GPU count becomes 2,083; scored count stays 1,867. It also refreshes three stale existing Radeon Pro V-series dump verification flags from the current seed records.
Closes #295
Source
Fetched live on 2026-09-29:
Validation
python -m app.validate: passed.overall: null.git diff --check: passed.No further complete, distinct models identified in this audit remain to add; incomplete/source-only rows need better sources rather than speculative fields.