CLI reports.h5: sedmlDataSetShapes per job, and Y curves labelled ::X

Aperta
#2,192 3 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
55/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
java
Ambito
cli, data

Direzione di ricerca

Start in vcell-cli/src/main/java/org/vcell/cli/run/results/NonSpatialResultsConverter.java around line 170, where one shapes list is shared across data sets and each job's flat size is appended. Then check the label format strings in ResultsConverter.java at lines 84 and 93, and the comment above add2DPlotsAsReports in prepareNonSpatialDataForHdf5. Done means one comma-joined shape per data set, curve data sets labelled without the ::X suffix, and the comment matching the BioSimulators-mode condition. Confirm the output still matches the legacy writer's expected reports.h5 on the UnifiedDataModel branch, since the issue asks for both changes to be deliberate.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Summary

Two places where the CLI's reports.h5 deviates from the BioSimulators conventions. Both are intentionally reproduced, for now, by the new data system on the UnifiedDataModel branch so its output can be checked byte-for-byte against the legacy writer. Fix them deliberately, in both places.

1. sedmlDataSetShapes has one entry per data set per job. NonSpatialResultsConverter.java:170 shares one shapes list across all data sets and appends each job's flat size to it. A 3-data-set report over a 3-value scan gets 9 entries such as "21", where the convention is one comma-joined shape per data set.

2. Plots reported as reports label every curve data set <plot>.<curve>::X. That includes the Y data generators: ResultsConverter.java:84 and line 93 use the same format string.

Related, minor: the comment above add2DPlotsAsReports in prepareNonSpatialDataForHdf5 says plots are added when not in BioSimulators mode, but the code adds them only in BioSimulators mode.

Lingua principale
Java
Stelle
140
Fork
35
Merge medio
17h 39m
PR unite (30g)
19

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di virtualcell/vcell

Tutte le issue di virtualcell/vcell

Issue simili

Altre issue su Java

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.