Inherited index early closes retain Chicago wall time in New York exchange calendars

オープン
#10,115 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

この issue はまだ評価されていません。

説明

priority:P1
Observed behavior

On master (7939b7878), the NDXP and NQX entries use America/New_York, but GetLastDailyMarketClose returns 12:00 Eastern on 2025-11-28. They inherit the wildcard index calendar's 12:00 America/Chicago early close without converting it into their exchange timezone. The inherited instant should be 13:00 Eastern.

Minimal reproduction, using the bundled market-hours database:

var expiry = new DateTime(2025, 11, 28);
var underlying = Symbol.Create("NDX", SecurityType.Index, Market.USA);
foreach (var ticker in new[] { "NDXP", "NQX" })
{
    var option = Symbol.CreateOption(underlying, ticker, Market.USA,
        OptionStyle.European, OptionRight.Call, 200m, expiry);
    var hours = MarketHoursDatabase.FromDataFolder()
        .GetExchangeHours(Market.USA, option, SecurityType.IndexOption);
    Console.WriteLine($"{ticker}: {hours.TimeZone}, {hours.GetLastDailyMarketClose(expiry, false):yyyy-MM-dd HH:mm}");
}

Observed for both: America/New_York, 2025-11-28 12:00.

Cause and impact

Common/Util/MarketHoursDatabaseJsonConverter.cs copies underlyingEntry.ExchangeHours.EarlyCloses directly at lines 307–309 and merges market-wide early closes at lines 338–340. The result is then constructed in the target ExchangeTimeZone at line 349. The NDX underlying itself inherits the wildcard Chicago index calendar, so the error also exists before the option entry inherits from NDX.

The affected calendar can mark the exchange closed an hour too early. A settlement helper that correctly preserves this calendar value still cannot establish the real exchange closing instant.

Required result

Inherited early closes must retain their intended instant when source and target exchange timezones differ, with tests covering NDX/NDXP/NQX and unchanged Chicago SPX/SPXW behavior. Review late-open inheritance for the same timezone assumption. A suggestion is to make the inheritance timezone-aware in the shared conversion path, rather than special-casing callers.

This calendar defect is present before #10114; that change's database-preservation guard does not introduce it.

主要言語
C#
スター
22.5k
フォーク
5.4k
平均マージ
22時間 6分
マージ済み PR(30日)
81

環境構築

Codespaces で開く

このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

QuantConnect/Lean のほかの issue

QuantConnect/Lean の issue をすべて見る

似ている issue

C# の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。