Skip to content

fix(data): resolve sourced Intel Atom placeholder on Zenfone 2 - #277

Merged
Seungpyo1007 merged 1 commit into
developfrom
fix/phone-placeholder-soc
Sep 25, 2026
Merged

Seungpyo1007 merged 1 commit into
developfrom
fix/phone-placeholder-soc

Conversation

@Seungpyo1007

@Seungpyo1007 Seungpyo1007 commented Sep 25, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Remap one Asus Zenfone 2 ZE551ML source variant from an ambiguous Intel Atom placeholder to the exact Z3580 SoC.
  • The record reports 4 GB RAM, and its raw chipset field explicitly assigns Z3580 to the 4 GB RAM model. Other placeholder references lack an unambiguous matching chip in their own sourced fields.

Mapping

Placeholder SoC Real SoC Smartphones
intel-atom-z3560-or-z3580 intel-atom-z3580 1

Remaining placeholder references

Placeholder SoC Smartphones
0 3610
1-0-ghz 118
1-0-ghz-cortex-a5 7
1-0-ghz-cortex-a8 14
1-2-ghz 13
1-2-ghz-cortex-a8 10
1-2-ghz-cortex-a9 2
1-22-ghz-cortex-a8 1
1-5-ghz 2
800-mhz 53
800-mhz-scorpion 2
alcatel-mobile-platform-2014 10
dual-core-1-0-ghz-cortex-a5 3
dual-core-1-0-ghz-cortex-a9 7
dual-core-1-2-ghz 106
dual-core-1-2-ghz-marvell 1
dual-core-1-3-ghz 35
dual-core-1-3-ghz-cortex-a7 9
dual-core-1-5-ghz 40
dual-core-1-5-ghz-cortex-a9 8
dual-core-1-6-ghz 3
dual-core-1-6-ghz-cortex-a9 12
exynos 100
huawei-mobile-platform-2011 30
intel-atom 5
intel-atom-x3 1
intel-atom-z3560-or-z3580 1
mediatek 58
mediatek-helio 245
nvidia-tegra-2 8
panasonic-mobile-platform-2011 5
pantech-mobile-platform-2008 3
quad-core 22
quad-core-1-0-ghz-cortex-a7 4
quad-core-1-1-ghz 13
quad-core-1-2-ghz 157
quad-core-1-2-ghz-cortex-a7 25
quad-core-1-2-ghz-cortex-a9 1
quad-core-1-3-ghz 146
quad-core-1-3-ghz-cortex-a7 125
quad-core-1-3-ghzquad-core-1-2-ghz-usa-model 1
quad-core-1-5-ghz 26
quad-core-1-5-ghz-cortex-a7 11
quad-core-1-6-ghz-cortex-a9 2
qualcomm 206
qualcomm-snapdragon 1700
qualcomm-snapdragon-s2 6
qualcomm-snapdragon-s3 42
qualcomm-snapdragon-s4-pro 39
spice-mobile-platform-2013 21
zte-mobile-platform-2012 8

Total remaining: 7077 smartphones across 51 placeholder slugs.

Validation

PYTHONIOENCODING=utf-8 python -m app.validate passed.

Refs #1

@github-actions github-actions Bot added bug Something isn't working data Dataset changes enhancement New feature or request labels Sep 25, 2026
@Seungpyo1007 Seungpyo1007 removed the enhancement New feature or request label Sep 25, 2026
@TechEngineBot

Copy link
Copy Markdown
Member

🔎 Data verification — Tier 0 (offline existence/trust)

Scored by app.verify; posted by TechEngineBot. Informational only — the structural gate (app.validate) is separate and authoritative for merge.

Changed records in this PR

38 record(s) scored.

%%{init: {"theme":"base","themeVariables":{"pie1":"#3fb950","pie2":"#d29922","pie3":"#f85149","pieStrokeWidth":"0px","pieOpacity":"1"}}}%%
pie showData
    title Verification bands — all records
    "Green" : 37
    "Yellow" : 1
    "Red" : 0
Loading
Category Distribution Total 🟢 🟡 🔴 🟢 %
smartphone 🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨 1 0 1 0 0.0%
cpu 🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩 37 37 0 0 100.0%
All 🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟨 38 37 1 0 97.4%

Full-dataset baseline

106328 record(s) scored.

%%{init: {"theme":"base","themeVariables":{"pie1":"#3fb950","pie2":"#d29922","pie3":"#f85149","pieStrokeWidth":"0px","pieOpacity":"1"}}}%%
pie showData
    title Verification bands — all records
    "Green" : 24158
    "Yellow" : 81074
    "Red" : 1096
Loading
Category Distribution Total 🟢 🟡 🔴 🟢 %
brand 🟩🟩🟩🟩🟩🟩🟩🟨🟨🟨🟨🟨 207 122 85 0 58.9%
soc 🟩🟨🟨🟨🟨🟨🟥🟥🟥🟥🟥🟥 2097 251 824 1022 12.0%
smartphone 🟩🟩🟨🟨🟨🟨🟨🟨🟨🟨🟨🟥 93396 17931 75397 68 19.2%
tablet 🟩🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟥 3455 291 3163 1 8.4%
watch 🟩🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟥 433 43 386 4 9.9%
pda 🟩🟩🟩🟩🟩🟩🟩🟨🟨🟨🟨🟨 140 83 57 0 59.3%
gpu 🟩🟩🟩🟩🟩🟩🟩🟨🟨🟨🟨🟨 2093 1274 819 0 60.9%
cpu 🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟨🟥 4507 4163 343 1 92.4%
All 🟩🟩🟩🟨🟨🟨🟨🟨🟨🟨🟨🟥 106328 24158 81074 1096 22.7%

Hard violations (forced red):

Count Check
1 !boost_ge_base

green = authoritative source + complete + consistent · yellow = plausible, needs confirmation · red = sparse/weak source or a hard contradiction. Promotion to verified runs in the scheduled verify-network workflow.

@Seungpyo1007
Seungpyo1007 merged commit a365e54 into develop Sep 25, 2026
5 checks passed
@Seungpyo1007
Seungpyo1007 deleted the fix/phone-placeholder-soc branch September 25, 2026 12:00
@github-project-automation github-project-automation Bot moved this from In Progress to Done in TechAPI-Project Sep 25, 2026
@TechEngineBot

Copy link
Copy Markdown
Member

TechEngine change review: PASS

Check Result
python -m app.validate PASS
New hard integrity anomalies vs PR base PASS

Changed data

Category Added Modified Deleted Added verified Added unverified Added Kaggle-sourced
brand 0 0 0 0 0 0
soc 0 0 0 0 0 0
smartphone 0 1 0 0 0 0
tablet 0 0 0 0 0 0
watch 0 0 0 0 0 0
pda 0 0 0 0 0 0
gpu 0 0 0 0 0 0
cpu 0 0 0 0 0 0

Changed record examples

smartphone modified

  • smartphone/asus/2015/zenfone-2-ze551ml/asus-zenfone-2-ze551ml-scrapegsma-2539.json - Asus Zenfone 2 ZE551ML

Heuristic review

  • Heuristic warnings: none found.

@TechEngineBot

Copy link
Copy Markdown
Member

TechEngine validation stats: PASS

Data summary

Category Total Verified Unverified Missing verified Tracked Verified % of tracked
brand 207 122 85 0 207 58.9%
soc 2097 193 1904 0 2097 9.2%
smartphone 93396 17501 75895 0 93396 18.7%
tablet 3455 273 3182 0 3455 7.9%
watch 433 39 394 0 433 9.0%
pda 140 74 66 0 140 52.9%
gpu 2093 1028 1065 0 2093 49.1%
cpu 4507 2869 1638 0 4507 63.7%
all 106328 22099 84229 0 106328 20.8%

Warning

Tracked verified coverage is below 50% for tablet 7.9% (273/3455), watch 9.0% (39/433), soc 9.2% (193/2097), smartphone 18.7% (17501/93396), all 20.8% (22099/106328), gpu 49.1% (1028/2093).
Tracked coverage excludes records missing the verified field; see the Missing verified column for those records.
This does not fail validation. Keep imported records verified: false until manual audit, but treat this as follow-up verification work before relying on the affected categories as curated data.

Validation notes

  • The integrity gate blocks only hard anomalies introduced by this PR relative to its merge base.
  • Full advisory outlier listings are suppressed on successful runs because they are dataset-wide and mostly stable between PRs.
  • Failure runs still include a detailed log excerpt for debugging.

Key output:

## app.validate
## integrity_check.py (PR head compared with PR base)
loaded CPU=4507 GPU=2093
Integrity section Flagged lines
structural 20
CPU name/tier consistency (desktop mainstream only) 0
CPU single>multi (cinebench/geekbench — should be multi>=single) 0
CPU era-vs-score outliers 8
CPU cross-source ratio outliers (possible wrong-variant) 155
GPU cross-source ratio outliers + sanity 18

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working data Dataset changes

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants