`ICA` does not surface whether the decomposition converged
メンテナーはふだん 1 日以内に返信
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 68/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 活発
- 技術スタック
- python
調査の方向性
Start at the ICA _fit entry point and trace the fastica, infomax, and picard paths, then inspect how save() serializes fitted state. Confirm how convergence is currently reported and identify the persistence and test locations. Done means fitted ICA objects expose the result and retain it after saving and loading, including when fitting stops at max_iter.
索引モデルが issue の本文から書いたものです。
説明
Summary
After
ica.fit(),ica.n_iter_holds the iteration count andica.max_iter
the limit, but there is no attribute saying whether fitting converged or
stopped at the limit. A user has to know to compare the two.Why it matters
These are different results and they are not distinguishable in saved output.
An ICA that hitmax_iterproduces components that are still used to pick
ica.exclude, and the exclusions are then applied to the data. Nothing
downstream — not the cleanedRaw, not a saved-ica.fif— records that the
decomposition was truncated.ica = mne.preprocessing.ICA(n_components=4, method='fastica', random_state=97, max_iter=200) ica.fit(raw) ica.n_iter_ == ica.max_iter # the only way to know, and easy not to checkMNE does warn at fit time, but a warning is not part of the object and does not
surviveica.save().Suggestion
A
converged_boolean set during_fit, and included in whatsave()writes,
would make this checkable after the fact. Happy to open a PR if that would be
welcome, including for theinfomaxandpicardpaths.Context
Noticed while writing an MNE adapter for
daftar. I am raising the same point with
three other packages that have the same pattern, so please read this as a
question about a shared convention rather than a complaint about MNE.
- 主要言語
- Python
- スター
- 3.6k
- フォーク
- 1.6k
- 平均マージ
- 1日 11時間
- マージ済み PR(30日)
- 82
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
mne-tools/mne-python のほかの issue
-
ENH: Add `description` filter parameter to `Raw.crop_by_annotations()`対応中かも @aman-coder03 が 203 日前に担当しました。 オープンENH
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
mne-tools/mne-python#13743 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
ENH
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
mne-tools/mne-python#14454 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
auto-issue
難易度 3/5 1〜2日 初心者へのやさしさ 48/100
mne-tools/mne-python#14442 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
mne-tools/mne-python#14428 · コメント 5 件 ·
メンテナーはふだん 1 日以内に返信
-
ENH
難易度 3/5 1〜2日 初心者へのやさしさ 58/100
mne-tools/mne-python#14427 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
mne-tools/mne-python の issue をすべて見る
似ている issue
-
area/config comp/tools P2 platform/telegram type/bug
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
NousResearch/hermes-agent#145412 ·
メンテナーはふだん 1 日以内に返信
-
[oblt-aw][autodoc] Update docs for security org scaffolding and Docs AI menu permission changes対応中かも @github-actions が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
メンテナーはふだん 1 日以内に返信
-
Community project: hosted MOSS-Image studio with a visual JSON layout builder (no ComfyUI needed)オープン
難易度 1/5 1〜3時間 初心者へのやさしさ 65/100
OpenMOSS/MOSS-Image#27 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
deepspeedai/DeepSpeed#8925 · コメント 1 件 ·
メンテナーはふだん 3 日以内に返信
-
external integration integration-submission
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
langchain-ai/docs#7077 ·
メンテナーはふだん 1 日以内に返信