Skip to content

Fix incorrect problem IDs - #340

Open
dweindl wants to merge 1 commit into
Benchmarking-Initiative:masterfrom
dweindl:fix-problem-ids
Open

dweindl wants to merge 1 commit into
Benchmarking-Initiative:masterfrom
dweindl:fix-problem-ids

Conversation

@dweindl

@dweindl dweindl commented Oct 22, 2026

Copy link
Copy Markdown
Member

Renames problems whose ID does not match the publication:

  • Giordano_Nature2020 → Giordano_NatMed2020 (published in Nature Medicine)
  • Schwen_PONE2014 → Schwen_PLoSOne2015 (published in 2015)
  • Weber_BMC2015 → Weber_BMCSystBiol2015 (BMC is the publisher, not the journal)

The new IDs use the ISO 4 journal abbreviation (as listed in PubMed). Other existing IDs use different abbreviations (e.g. PNAS, MSB, JBC, CellSystems); they are not wrong and are not changed here.

The SBML model IDs and names are updated accordingly. The former IDs are still accepted by get_problem, get_problem_yaml_path, get_simulation_df (v1 and v2), and get_metadata, with a FutureWarning; they are listed in RENAMED_MODELS, but are not part of MODELS. Links to the old directories break.

Closes #170.

Overlaps with #339: whichever is merged second needs the references in the renamed metadata.yaml files updated (BibTeX key = problem ID).

🤖 Generated with Claude Code

Rename problems whose ID does not match the publication:

* `Giordano_Nature2020` -> `Giordano_NatMed2020` (published in Nature
  Medicine)
* `Schwen_PONE2014` -> `Schwen_PLoSOne2015` (published in 2015)
* `Weber_BMC2015` -> `Weber_BMCSystBiol2015` (BMC is the publisher,
  not the journal)

The new IDs use the ISO 4 journal abbreviation (as listed in PubMed).
Other existing IDs use different abbreviations (e.g. `PNAS`, `MSB`,
`JBC`); they are not wrong and are not changed here.

The SBML model IDs and names are updated accordingly. The former IDs
are still accepted by the accessor functions with a `FutureWarning`
(`RENAMED_MODELS`), but are not part of `MODELS`.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@dweindl
dweindl marked this pull request as ready for review October 22, 2026 12:47

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

mislabeling Giordano_Nature2020

1 participant