fix: remove stale Wikidata facts, fix Meta AI valuation/cash-burn labels#3248
fix: remove stale Wikidata facts, fix Meta AI valuation/cash-burn labels#3248
Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (5)
📝 WalkthroughWalkthroughThese changes remove and update facts from multiple YAML company data files within the factbase package. Specifically, Wikidata-sourced facts are removed from Anthropic, DeepMind, and OpenAI records. A Reality Labs financial property in Meta AI's data is updated from Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Suggested labels
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@packages/factbase/data/things/meta-ai.yaml`:
- Around line 69-74: Add a new property definition for "cumulative-losses" to
the properties schema so the YAML entry in
packages/factbase/data/things/meta-ai.yaml validates and renders correctly: open
packages/factbase/data/properties.yaml and add an entry keyed
"cumulative-losses" mirroring the financial formatting used by "cash-burn" (use
the same type/format/units/label fields as "cash-burn" to ensure currency/number
formatting and display), include a brief description and any relevant aliases,
and commit the change; ensure the property name exactly matches
"cumulative-losses".
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 61457fd7-163d-42ca-9949-8fd46c85f454
📒 Files selected for processing (4)
packages/factbase/data/things/anthropic.yamlpackages/factbase/data/things/deepmind.yamlpackages/factbase/data/things/meta-ai.yamlpackages/factbase/data/things/openai.yaml
💤 Files with no reviewable changes (3)
- packages/factbase/data/things/deepmind.yaml
- packages/factbase/data/things/anthropic.yaml
- packages/factbase/data/things/openai.yaml
Test Coverage Report
Coverage is advisory only — no thresholds enforced. Generated by |
9b42db9 to
b9fd2bb
Compare
Bug 47: Remove AMI Labs $3.5B valuation description from meta-ai.yaml — that valuation belongs to Yann LeCun's separate AMI Labs startup, not Meta AI. Bug 48: Rename cash-burn → cumulative-losses for Reality Labs $70B figure in meta-ai.yaml; the note now explicitly states this is a ~5-year cumulative total, not an annual figure. Bug 56: Remove stale Wikidata-imported facts superseded by more recent data: - anthropic.yaml: headcount=500 (CTxrfaCzbw); current value is 4,074 - openai.yaml: headcount=375 (JU8dVl7ViQ); current value is 4,500 - openai.yaml: revenue=5363248 (hf1lYXbYXQ); ambiguous unit/date, stale - openai.yaml: legal-structure=501(c)(3) (Jjf6iAusCA); OpenAI is now a PBC - deepmind.yaml: headcount=10000 (mZo3jiRTlQ); current value is 6,000 Bug 57: Remove duplicate facts: - deepmind.yaml: duplicate founded-date 2010-01 (F2JXn1Rz4g) removed; keep the Wikipedia-sourced 2010-09 with specific month - openai.yaml: duplicate total-funding=5.79e+10 at 2025-03 (f_m5b6R9pIby) removed; keep the first entry with the more detailed SoftBank round notes Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Mirrors cash-burn format (financial, USD, billions display). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
b9fd2bb to
2960ffc
Compare
Summary
Fixes 4 P2 bugs from QA sweep (Discussion #3220):
cash-burn→cumulative-lossesfor Meta AI's $70B Reality Labs figure (was mislabeled as annual)7 stale/incorrect facts removed, 1 property renamed.
Test plan
🤖 Generated with Claude Code
Summary by CodeRabbit