Repository navigation
fix: map renamed yfinance ticker PSKY -> SKYD - #352
Merged
Merged
Conversation
Paramount Skydance changed its name to Skydance Corporation and moved its Class B stock from Nasdaq (`PSKY`) to the NYSE as `SKYD`, effective 2026-10-06, when its acquisition of Warner Bros. Discovery closed. It is the same security under a new symbol, so this is a rename, not a delisting: https://www.sec.gov/Archives/edgar/data/0002041610/000110465926113076/tm2626659d3_8k.htm ; https://www.tradingview.com/news/tradingview:01ced82c4b7be:0-skydance-to-list-class-b-shares-and-warrants-on-nyse-ticker-to-change-to-skyd/ The nightly fetch flagged `PSKY` as uncurated (no data, not in the S&P 500 list). Add a `ticker_renames` entry so the updater fetches history under `SKYD` and writes it to `PSKY`'s resolution file, matching the existing `FI`/`MMC`/`BK`/`SATS`/`EQR` handling. Extend `test_known_renames_present` with the new mapping. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_012pDUkRhKXgpLripTZNtGwp
houtanb
force-pushed
the
claude/psky-yfinance-fix
branch
from
November 6, 2026 15:45
1e5512c to
9c56c0f
Compare
houtanb
marked this pull request as ready for review
November 6, 2026 15:45
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.
Requested by Houtan · Slack thread
Before: The nightly yfinance fetch (2026-11-01) flagged
PSKYas an uncurated ticker: it returns no data and is no longer in the S&P 500 list. It is in neithernullified_questionsnorticker_renames. Each night it gets carried forward as resolved and triggers the Slack alert. Its resolution file also stops getting new prices, so thePSKYpair questions in the pool would resolve to stale prices.After:
PSKYmaps toSKYDin yfinanceticker_renames. The fetch no longer requestsPSKYand fetchesSKYDinstead. The updater writesSKYD's history toPSKY's resolution file, so existingPSKYquestions and pairs resolve on live prices and the alert stops.test_known_renames_presentnow also assertsPSKY->SKYD.How: this is a rename, not a delisting. When Paramount Skydance's acquisition of Warner Bros. Discovery closed on 2026-10-06, the company changed its name to Skydance Corporation. Its Class B stock moved from Nasdaq (
PSKY) to the NYSE under the new symbolSKYD. The security and its holders stayed the same; only the symbol changed. (The cash-out delisting was WBD's, handled in #341.) Sources:PSKYtoSKYDwith the NYSE listing on 2026-10-06, and the name change to Skydance Corporation: https://www.sec.gov/Archives/edgar/data/0002041610/000110465926113076/tm2626659d3_8k.htm''https://cordcuttersnews.com/its-done-paramount-completes-warner-bros-discovery-acquisition-launches-combined-company-as-skydance''/Caveats for the reviewer:
PSKYorSKYDfrom the sandbox, because its egress proxy blocks the Yahoo Finance hosts (403 on CONNECT). I also could not check that Yahoo serves pre-2026-10-06PSKYhistory underSKYD. Please confirm thatyf.Ticker("SKYD").history(period="max")goes back to Aug 2025 before merging.SKYD WS) to Class B holders. Yahoo may adjust historical closes for that distribution.src/sources/yfinance_questions.pystill listsPSKY. With the rename, those pairs drop out of sampling but keep resolving. Regenerating the pool would pick upSKYDif wanted.Tests:
ruff checkandruff format --checkare clean on the touched files.pytestonsrc/tests/unit/sources/test_yfinance.py,test_yfinance_pairs.py,src/tests/unit/scripts/test_generate_yfinance_pairs.pyandsrc/tests/unit/test_base_source.pygives 154 passed. The full suite needs Python 3.14 syntax, which the sandbox cannot install.Generated by Claude Code