The band-edge term that floors comparisons between windows is recorded only in an archived SEP

オープン 初心者向け
#239 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

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

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

評価

難易度
2/5
見積もり時間
半日
初心者へのやさしさ
62/100
issue の種類
ドキュメント
明瞭さ
おおむね明確
活発さ
活発
技術スタック
python
領域
documentation

調査の方向性

docs/dev/seps/archive/sep-0021-absorbing-boundaries.md の引用箇所(およそ 2030、2036、2074、2194 行目付近)から読み始め、バンド端という用語とその図を理解してください。次に、docs/dev/accuracy.md と docs/guide/numerics/boundaries.md のウィンドウ確認に関する案内に、図と SEP へのリンクを添えた短いセクションを追加してください。両方のページにこの内容が記載され、ドキュメントがビルドできれば完了です。メンテナーから依頼がない限り、任意の collar/mask 測定には手を付けないでください。

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

説明

area: boundaries documentation priority: low
What is missing

Comparing runs on different time windows has an error floor that has nothing to do with the boundary, and it is recorded only in an archived SEP. SEP 21's phase 0a found that wide-window references converge only at first order in the window factor. The term is Kerr-generated content at the band's low-frequency edge, where the spectral collar and the mask are sampled on bins that move with the bin spacing. Figures from that SEP, not re-run:

Neither accuracy.md nor the boundaries guide states this (searched for "band-edge", "mesh term" and "window factor").

Why it matters

Refining a window is how a user tests a window, and the tutorial teaches it. Since fft_size="auto" makes grids differ between neighbouring requests, a user who does this meets a first-order error and may take it for a poor collar. SEP 21 worked around it by extrapolating references in the window's inverse, and comparing with and without temporal absorption at equal window.

To do
  • Add it to accuracy.md and to the boundaries guide's advice on checking a window, saying to compare with and without the temporal collar at the same grid, or to extrapolate as SEP 21 did.
  • Optional, and unchecked: measure whether a collar and mask defined so that the sampled edge does not move with the spacing remove the first-order term. Change nothing without that measurement. A change here would touch every run's numbers, so it is a maintainer's call.
Context

#227 needs to separate its effect from this one. #229 covers the size choice.

Done when

Both pages state it with the SEP's figures and a link, and the docs build. The second box, if taken, posts its measurement here.


Checked against commit 8a9f764.

主要言語
Python
スター
3
フォーク
3
平均マージ
8時間 37分
マージ済み PR(30日)
159

環境構築

はじめの一歩

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

LupoLab/solprop のほかの issue

LupoLab/solprop の issue をすべて見る

似ている issue

Python の issue をもっと見る

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

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