CLI reports.h5: wrong repeated-task values and parameter names for multi-parameter scans
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
Start in vcell-cli/src/main/java/org/vcell/cli/run/results/NonSpatialResultsConverter.java around lines 144-150, where the repeated-task names and values are built, then compare with getScanBounds() and getConstantArraySpec(name) in vcell-core's MathOverrides.java to see how jobs are numbered. Done means each constant's values are listed in the same sorted order as the HDF5 dimensions, and names, values and dimensions agree for a scan of two or more parameters. No existing test covers a two-parameter scan, so a new one is needed.
索引モデルが issue の本文から書いたものです。
説明
Summary
For a simulation with two or more scanned constants, the CLI writes wrong repeated-task metadata into the BioSimulators reports.h5. The data itself is laid out correctly. Single-parameter scans are unaffected.
Two defects
1. Wrong values in sedmlRepeatedTaskValues. NonSpatialResultsConverter.java:150 fills each constant's value list with getActualExpression(name, new ScanIndex(i)). ScanIndex is a job index, not a position within that constant's own values. MathOverrides numbers jobs row-major over the sorted constant names, first one outermost (MathOverrides.java:186), so job i only lines up with value i when one constant is scanned.
Example: scanning Ka = [0.5, 5] and kOn = [10, 20, 30] gives jobs 0 and 1 at coordinates (0,0) and (0,1). Ka is reported as [0.5, 0.5] instead of [0.5, 5].
2. Parameter names in hash order. sedmlRepeatedTaskParameterNames comes from getScannedConstantNames() (line 144), which reads from a HashMap (MathOverrides.java:47). The scan bounds, and therefore the HDF5 dimensions and data, use sorted order (getScanBounds()). When the two orders differ, names, value lists and dimensions refer to different parameters.
Fix
Take each constant's own values, getConstantArraySpec(name).getConstants(), and list constants in sorted order. The new data system (org.vcell.data, on the UnifiedDataModel branch) does this in SedmlReportDatasets.scanDimensions, with a test against MathOverrides' job numbering. No test archive scans two parameters, so neither system is exercised on this end to end.
- 主要言語
- Java
- スター
- 140
- フォーク
- 35
- 平均マージ
- 22時間 6分
- マージ済み PR（30日）
- 14
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
virtualcell/vcell のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
virtualcell/vcell#2193 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Accessibility Requirement Java-based User Interface
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
virtualcell/vcell#2138 ·
メンテナーはふだん 1 日以内に返信
-
Accessibility Requirement Java-based
難易度 2/5 1〜3時間 初心者へのやさしさ 80/100
virtualcell/vcell#2137 ·
メンテナーはふだん 1 日以内に返信
-
Accessibility Requirement User Interface
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
virtualcell/vcell#2135 ·
メンテナーはふだん 1 日以内に返信
-
Accessibility Requirement Java-based User Interface
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
virtualcell/vcell#2134 ·
メンテナーはふだん 1 日以内に返信
virtualcell/vcell の issue をすべて見る
似ている issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 68/100
Vakore/ZappierGames#285 ·
-
Type: Possible bug
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
AntennaPod/AntennaPod#8895 ·
メンテナーはふだん 1 日以内に返信
-
bug build failed
難易度 1/5 1時間未満 初心者へのやさしさ 91/100
objectionary/eo#9852 · コメント 1 件 ·
メンテナーはふだん 2 日以内に返信
-
dependencies enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
BrendonButler/Shops#79 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
jdsalasca/open-civic-signal-os#144 ·
メンテナーはふだん 1 日以内に返信