Repository navigation
Fix tablet inventory dates and fields from the purchase records - #62
Merged
Merged
Conversation
Reconciled against the private purchase records on 2026-10-05. - 110 OrderDates from the 2026-04-01 bulk import were one day early (probably a time-zone shift); each moves forward one day - WAT.0014 ModelName SD-501C -> SD-510C - WAT.0076 Vendor WACOM -> AMAZON - WAT.0083 OrderDate 2025-08-15 -> 2026-08-15 - SAT.0001 OrderDate 2024-10-03 -> 2024-09-24 - XET.0004 Vendor and OrderDate filled in (AMAZON, 2025-07-01) - XPT.0015 OrderDate filled in (2025-10-21) - HUT.0019 TabletType PENTABLET -> PENDISPLAY (the tablet record already says PENDISPLAY) - UGT.0001 Vendor UNKNOWN -> UGEE, OrderDate UNKNOWN -> 2023-12-01 - Galaxy Book5 Pro 360: Samsung's SKU NP960QHA-KG1US added as an alternate name; Model.Id stays Galaxy-Book5-Pro-360 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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
1 participant
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.
Before: 110 tablet units from the 2026-04-01 bulk import had an OrderDate one day early, probably from a time-zone shift during the import. Eight more units had a wrong or missing field: a swapped model name, wrong vendors and dates, blank or UNKNOWN values, and a pen display recorded as a pen tablet.
After: those values now match the private purchase records, which were checked line by line on 2026-10-05.
NP960QHA-KG1USas an alternate name. Its Model.Id staysGalaxy-Book5-Pro-360, so SAT.0001's ModelId and every EntityId are unchanged.How: a one-shot script checked each unit's current value against the expected "before" value, and checked that every date moved exactly one day, before writing anything through
commitDatasetUpdate. Data-quality, format-data and generate are clean. The import script that caused the shift was never committed, so there is nothing in the repo to fix. Pen inventory has no order dates.🤖 Generated with Claude Code