Authored measures' most-recent sort: mixed timezone offsets and same-instant ties still depend on order
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 35/100
- issue の種類
- バグ
- 明瞭さ
- 説明が足りない
- 活発さ
- 活発
- 技術スタック
- typescript
- 領域
- data
調査の方向性
Start with the review in #749 and trace the authored-measure sorting path, including normalize.ts and the cql-execution comparison behavior. Confirm how authored cms122 handles mixed offsets and same-instant ties, then get the owner’s decision on the tie rule before defining completion.
索引モデルが issue の本文から書いたものです。
説明
Two ordering edge cases are left after #749. Both affect only the authored measures; the official CMS measures don't use this sort.
1. Mixed timezone offsets order by text
cql-execution compares sort keys with <. A CQL DateTime has no valueOf, so it compares the toString() output, which ends in the offset. Results with the same offset sort correctly; results with different offsets can sort wrongly when they're close in time.
- Example (authored cms122): 7.5% at
2026-05-13T23:00:00-10:00(the newer result) against 10.5% at2026-05-14T02:00:00+00:00gives OVERDUE in both orders. - How it can happen: WebChart-sourced times carry a local offset (Maui is −10:00), while
normalize.tsfalls back toissued, usually inZ, wheneffectiveDateTimeis missing. - Impact: at most one day on the day-window measures.
- Fix options: normalise instants to UTC at ingest, or give the sort key one offset.
2. Ties keep input order
Two results at the same instant compare as "greater" both ways, so Last() picks the first-listed of the tied pair. That matters only for value-reading measures (authored cms122: 7.5 then 10.5 gives COMPLIANT; 10.5 then 7.5 gives OVERDUE). A tie needs a rule: a secondary key such as issued, or worst-value-wins. That choice is clinical, so it's an owner decision.
Found in #749's review. Low priority.
- 主要言語
- TypeScript
- スター
- 0
- フォーク
- 0
- 平均マージ
- 2時間 51分
- マージ済み PR（30日）
- 109
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
Taleef7/workwell のほかの issue
-
documentation owner-ops waiting
難易度 1/5 1〜3時間 初心者へのやさしさ 86/100
メンテナーはふだん 1 日以内に返信
-
owner-ops waiting
難易度 1/5 1〜3時間 初心者へのやさしさ 85/100
メンテナーはふだん 1 日以内に返信
-
Measure labels carry no version outside the measure page, and several screens print a version that never ran対応中かも @Taleef7 が今日担当しました。 オープンmaui-pilot pilot-trust
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
メンテナーはふだん 1 日以内に返信
-
infra maui-pilot
難易度 3/5 1〜2日 初心者へのやさしさ 65/100
メンテナーはふだん 1 日以内に返信
-
cql-engine pilot-trust
難易度 3/5 1〜2日 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
Taleef7/workwell の issue をすべて見る
似ている issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
MystenLabs/MemWal#1163 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
Mondriaan
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
knaw-huc/textannoviz#709 ·
メンテナーはふだん 1 日以内に返信
-
billion-context-pi
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
ranxianglei/billion-context#2521 · コメント 3 件 ·
メンテナーはふだん 1 日以内に返信
-
Add: YRF Music Nepalオープンstreams:add
難易度 1/5 1時間未満 初心者へのやさしさ 62/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100