Skip to content

Commit 20fc213

Browse files
authored
Merge pull request #152 from GetTechAPI/develop
release: verification 19.97% per record / 33.69% per product
2 parents e3e8be6 + 461de8b commit 20fc213

7,193 files changed

Lines changed: 127258 additions & 117313 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.github/workflows/bump-engine.yml

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,13 @@ jobs:
3232
- name: Update TechEngine submodule to latest main
3333
id: bump
3434
run: |
35+
# git-flow: the bump is integration work, so it lands on develop.
36+
# main only moves through a release PR. Switch branch BEFORE touching
37+
# the submodule so the change is staged on the branch it will be
38+
# committed to.
39+
git fetch origin develop
40+
git checkout -B develop origin/develop
41+
git submodule update --init --recursive TechEngine
3542
git submodule update --remote --recursive TechEngine
3643
if git diff --quiet -- TechEngine; then
3744
echo "changed=false" >> "$GITHUB_OUTPUT"
@@ -51,8 +58,8 @@ jobs:
5158
git config user.email "289859915+TechEngineBot@users.noreply.github.com"
5259
git add TechEngine
5360
git commit -m "chore: bump TechEngine submodule to ${NEW_SHA}"
54-
git pull --rebase origin main
55-
git push origin HEAD:main
61+
git pull --rebase origin develop
62+
git push origin HEAD:develop
5663
echo "techapi_sha=$(git rev-parse HEAD)" >> "$GITHUB_OUTPUT"
5764
5865
- name: Dispatch reciprocal TechAPI bump to TechEngine

.github/workflows/validate-data.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ on:
88
- "data/**"
99
- "app/validate.py"
1010
push:
11-
branches: [main]
11+
branches: [main, develop]
1212
paths:
1313
- "data/**"
1414
- "app/validate.py"

README.md

Lines changed: 18 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -46,12 +46,26 @@ python -m app.validate
4646

4747
The validator uses only the Python standard library; no install step required.
4848

49+
## Branching
50+
51+
This repo follows git-flow:
52+
53+
| Branch | Role |
54+
| --- | --- |
55+
| `main` | Released state. Publishing the site and notifying TechEngine both fire on a push here, so it only moves through a release PR. |
56+
| `develop` | Integration. Every data, feature and fix branch targets this. |
57+
| `feat/*`, `data/*`, `fix/*`, `chore/*` | Short-lived work branches, cut from `develop`. |
58+
59+
A release is a PR from `develop` to `main`; merging it deploys the site and tells
60+
TechEngine to pick up the new data. Automation follows the same rule — the
61+
submodule bump and the verified-promotion bot both land on `develop`.
62+
4963
## Contributing
5064

51-
Open a PR with the new/updated JSON file. The PR template walks through what to
52-
include. The validator must pass. All records (`brand`, `soc`, `smartphone`,
53-
`gpu`, and `cpu`) must include `source_urls` with at least one canonical
54-
reference (vendor product page, Wikipedia infobox, datasheet).
65+
Open a PR **against `develop`** with the new/updated JSON file. The PR template
66+
walks through what to include. The validator must pass. All records (`brand`,
67+
`soc`, `smartphone`, `gpu`, and `cpu`) must include `source_urls` with at least
68+
one canonical reference (vendor product page, Wikipedia infobox, datasheet).
5569

5670
## License
5771

data/_verify/ledger.jsonl

Lines changed: 6337 additions & 0 deletions
Large diffs are not rendered by default.

data/_verify/state/crossref_cache.jsonl

Lines changed: 2069 additions & 0 deletions
Large diffs are not rendered by default.

data/_verify/state/scores.jsonl

Lines changed: 105742 additions & 105742 deletions
Large diffs are not rendered by default.

data/_verify/state/url_cache.jsonl

Lines changed: 3330 additions & 3136 deletions
Large diffs are not rendered by default.

data/_verify/status.json

Lines changed: 74 additions & 47 deletions
Original file line numberDiff line numberDiff line change
@@ -1,87 +1,114 @@
11
{
2-
"generated_at": "2026-07-31T09:27:33Z",
2+
"generated_at": "2026-08-06T03:22:54Z",
33
"schema": 1,
44
"totals": {
55
"records": 105742,
6-
"verified": 14777,
7-
"verified_pct": 13.97,
8-
"green": 14778,
9-
"yellow": 89175,
10-
"red": 1789,
11-
"promotable": 14778
6+
"verified": 21114,
7+
"verified_pct": 19.97,
8+
"models": 62148,
9+
"verified_models": 20936,
10+
"verified_models_pct": 33.69,
11+
"green": 21110,
12+
"yellow": 83431,
13+
"red": 1201,
14+
"promotable": 21110
1215
},
1316
"by_category": {
1417
"brand": {
1518
"total": 207,
1619
"verified": 24,
1720
"verified_pct": 11.59,
21+
"models": 207,
22+
"verified_models": 24,
23+
"verified_models_pct": 11.59,
1824
"green": 24,
1925
"yellow": 183,
2026
"red": 0,
2127
"promotable": 24
2228
},
2329
"soc": {
2430
"total": 2104,
25-
"verified": 123,
26-
"verified_pct": 5.85,
27-
"green": 123,
28-
"yellow": 804,
29-
"red": 1177,
30-
"promotable": 123
31+
"verified": 146,
32+
"verified_pct": 6.94,
33+
"models": 2104,
34+
"verified_models": 146,
35+
"verified_models_pct": 6.94,
36+
"green": 146,
37+
"yellow": 844,
38+
"red": 1114,
39+
"promotable": 146
3140
},
3241
"smartphone": {
3342
"total": 93396,
34-
"verified": 11602,
35-
"verified_pct": 12.42,
36-
"green": 11602,
37-
"yellow": 81206,
38-
"red": 588,
39-
"promotable": 11602
43+
"verified": 17384,
44+
"verified_pct": 18.61,
45+
"models": 52309,
46+
"verified_models": 17237,
47+
"verified_models_pct": 32.95,
48+
"green": 17382,
49+
"yellow": 75934,
50+
"red": 80,
51+
"promotable": 17382
4052
},
4153
"tablet": {
4254
"total": 3455,
43-
"verified": 174,
44-
"verified_pct": 5.04,
45-
"green": 174,
46-
"yellow": 3266,
47-
"red": 15,
48-
"promotable": 174
55+
"verified": 218,
56+
"verified_pct": 6.31,
57+
"models": 1216,
58+
"verified_models": 192,
59+
"verified_models_pct": 15.79,
60+
"green": 218,
61+
"yellow": 3236,
62+
"red": 1,
63+
"promotable": 218
4964
},
5065
"watch": {
5166
"total": 433,
52-
"verified": 11,
53-
"verified_pct": 2.54,
54-
"green": 11,
55-
"yellow": 415,
56-
"red": 7,
57-
"promotable": 11
67+
"verified": 18,
68+
"verified_pct": 4.16,
69+
"models": 193,
70+
"verified_models": 18,
71+
"verified_models_pct": 9.33,
72+
"green": 18,
73+
"yellow": 410,
74+
"red": 5,
75+
"promotable": 18
5876
},
5977
"pda": {
6078
"total": 140,
61-
"verified": 27,
62-
"verified_pct": 19.29,
63-
"green": 27,
64-
"yellow": 112,
65-
"red": 1,
66-
"promotable": 27
79+
"verified": 66,
80+
"verified_pct": 47.14,
81+
"models": 112,
82+
"verified_models": 61,
83+
"verified_models_pct": 54.46,
84+
"green": 66,
85+
"yellow": 74,
86+
"red": 0,
87+
"promotable": 66
6788
},
6889
"gpu": {
6990
"total": 2030,
70-
"verified": 244,
71-
"verified_pct": 12.02,
72-
"green": 245,
73-
"yellow": 1785,
91+
"verified": 663,
92+
"verified_pct": 32.66,
93+
"models": 2030,
94+
"verified_models": 663,
95+
"verified_models_pct": 32.66,
96+
"green": 664,
97+
"yellow": 1366,
7498
"red": 0,
75-
"promotable": 245
99+
"promotable": 664
76100
},
77101
"cpu": {
78102
"total": 3977,
79-
"verified": 2572,
80-
"verified_pct": 64.67,
81-
"green": 2572,
82-
"yellow": 1404,
103+
"verified": 2595,
104+
"verified_pct": 65.25,
105+
"models": 3977,
106+
"verified_models": 2595,
107+
"verified_models_pct": 65.25,
108+
"green": 2592,
109+
"yellow": 1384,
83110
"red": 1,
84-
"promotable": 2572
111+
"promotable": 2592
85112
}
86113
}
87114
}

data/cpu/amd/2009/consumer/amd-athlon-7450-dual-core.json

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,8 @@
3030
"cinebench_r11_5_multi": null,
3131
"cinebench_r10_single": null,
3232
"cinebench_r10_multi": null,
33-
"passmark_single": null,
34-
"passmark_cpu_mark": null,
33+
"passmark_single": 939,
34+
"passmark_cpu_mark": 872,
3535
"specint2006": null,
3636
"specfp2006": null,
3737
"dhrystone_mips": null,
@@ -40,6 +40,7 @@
4040
"msrp_usd": null,
4141
"verified": false,
4242
"source_urls": [
43-
"https://www.kaggle.com/datasets/lincolnzh/cpu-specifications-dataset"
43+
"https://www.kaggle.com/datasets/lincolnzh/cpu-specifications-dataset",
44+
"https://www.cpubenchmark.net/cpu.php?cpu=AMD+Athlon+7450"
4445
]
4546
}

data/cpu/amd/2009/consumer/amd-athlon-7550-dual-core.json

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -30,16 +30,17 @@
3030
"cinebench_r11_5_multi": null,
3131
"cinebench_r10_single": null,
3232
"cinebench_r10_multi": null,
33-
"passmark_single": null,
34-
"passmark_cpu_mark": null,
33+
"passmark_single": 969,
34+
"passmark_cpu_mark": 977,
3535
"specint2006": null,
3636
"specfp2006": null,
3737
"dhrystone_mips": null,
3838
"whetstone_mflops": null,
3939
"superpi_1m_sec": null,
4040
"msrp_usd": null,
41-
"verified": false,
41+
"verified": true,
4242
"source_urls": [
43-
"https://www.kaggle.com/datasets/lincolnzh/cpu-specifications-dataset"
43+
"https://www.kaggle.com/datasets/lincolnzh/cpu-specifications-dataset",
44+
"https://www.cpubenchmark.net/cpu.php?cpu=AMD+Athlon+7550"
4445
]
4546
}

0 commit comments

Comments
 (0)