PEtab-dev / PEtab-dev/libpetab-python

Improve model abstractions

オープン
#518 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

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

主要言語
Python
スター
18
フォーク
9
PR マージ指標
30日以内にマージされた PR はありません

説明

The current model abstraction in petab.v1.models.model.Model isn't great. Non-exhaustive list of problems:

  • We don't have proper handling of derived parameters (#517 / https://github.com/PEtab-dev/libpetab-python/pull/517#issuecomment-5566488336)
  • is_state_variable conflates different questions. The simulator decides what's treated as a state variable - that's none of PEtab's business.
  • Ideally, the library would more closely adopt the specs' taxonomy regarding constant, differential, and algebraic entities.

Improve that. This will probably impact quite a number of places and needs to be addressed carefully.

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

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

調査の方向性

Start by reading petab.v1.models.model.Model and the discussion in #517, including the linked pull request comment. Map how derived parameters and is_state_variable are used across the package before proposing changes. Done means the model abstraction reflects the constant, differential, and algebraic taxonomy without conflating simulator decisions.

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

評価

技術スタック
python
領域
tooling
issue の種類
リファクタリング
難易度
5/5
見積もり時間
1週間以上
活発さ
活発
明瞭さ
説明が足りない
初心者へのやさしさ
25/100

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

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