Plot does not appear in some cases if rangebreaks overlap

オープン

#7,270 2024年11月14日に作成コメント 3 件リアクション 0 件担当者 0 名 GitHub で見る

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

評価

難易度: 3/5
見積もり時間: 1〜2日
初心者へのやさしさ: 68/100

issue の種類: バグ
明瞭さ: おおむね明確
活発さ: 活発
技術スタック: javascript
領域: data-visualization

調査の方向性

リンクされているCodePenの再現から始め、src/plots/cartesian/autorange.jsの199行目付近にあるrangebreakの処理を調査してください。同様のissue #5783と動作を比較し、holidayとbusiness-dayのrangebreakが重なった場合でも、無効なbreak lengthを生成せずにチャートがレンダリングされることを確認してください。

索引モデルが issue の本文から書いたものです。

説明

bug P2

I've created a Codepen for the issue: https://codepen.io/mofojed/pen/ZEgPJPN
In this case, the data spans a week, and range breaks are specified for a full holiday on Thursday, Nov 23, 2023 (Thanksgiving), and a half day on Friday, Nov 24, 2023. These specifications overlaps with other patterns for business days.
The chart does not appear. It looks like the rangebreaks that Plotly.js locates for the plot is:

Where some of those overlap. It looks like that isn't taken into account when calculating the breaks length, so when there's overlap it will not generate a valid break length: https://github.com/plotly/plotly.js/blob/29ca37b4ba3ee28a3ffe3d48eab97a080d602af3/src/plots/cartesian/autorange.js#L199
I think it would make sense to check for overlapping here. I'll see if it's possible to adjust holidays to not overlap with any existing business periods as well...

#5783 is similar

主要言語: JavaScript
スター: 18.3k
フォーク: 2k
平均マージ: 2日 10時間
マージ済み PR（30日）: 30

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

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

plotly/plotly.js のほかの issue

[CHORE]: Warn on config values that do not match the schema オープン

chore P3 plotly-internal size: 3 task

難易度 2/5 1〜3時間初心者へのやさしさ 77/100

plotly/plotly.js#8064 · 2026年9月21日に作成
[CHORE]: Change default to `true` for `responsive` config attribute オープン

chore P1 plotly-internal size: 1 task

難易度 1/5 1時間未満初心者へのやさしさ 82/100

plotly/plotly.js#8034 · 2026年9月10日に作成
[CHORE]: Deprecate plotly.js-mapbox* npm packages オープン

chore P3 plotly-internal size: 1 task

難易度 2/5 1〜3時間初心者へのやさしさ 65/100

plotly/plotly.js#8029 · 2026年9月8日に作成
[mkdocs] Update link in Edit this Page button オープン

bug

難易度 2/5 1〜3時間初心者へのやさしさ 65/100

plotly/plotly.js#7648 · コメント 3 件 · 2025年11月18日に作成
BUILDING.md: minor typo and some Markdown syntax errors オープン

bug infrastructure P2

難易度 1/5 1時間未満初心者へのやさしさ 65/100

plotly/plotly.js#6930 · 2024年3月15日に作成

plotly/plotly.js の issue をすべて見る

似ている issue

P4 FR: Add one-click copy-to-clipboard button to code snippets in static workshop HTML オープン

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

palladius/rails8-app-on-gcp#145 · 2026年9月23日に作成
Disable auto-cloaking not working オープン

難易度 2/5 1〜3時間初心者へのやさしさ 65/100

dotenvx/dotenv-vscode#139 · 2026年9月23日に作成
Remove sw.https.window.html and sw-416.https.window.html from Interop 2026 fetch オープン

test-change-proposal

難易度 2/5 1〜3時間初心者へのやさしさ 65/100

web-platform-tests/interop#1455 · 2026年9月23日に作成
Missing security headers on corsair.dev (CSP, X-Frame-Options, etc.) オープン

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

corsairdev/corsair#1764 · 2026年9月23日に作成
Empty .ts route file is scaffolded with JSX, breaking all route generation オープン

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

TanStack/router#8480 · 2026年9月23日に作成

JavaScript の issue をもっと見る