PEtab-dev / PEtab-dev/petab_sciml

Linter

Open
#69 7 comments 0 reactions 1 assignee View on GitHub

@m-philipps is already working on this.

Since Apr 14, 2026.

Dominant language
Python
Stars
11
Forks
4
PR merge metrics
No merged PRs in 30d

Description

A list of things that should be checked by extending the libpetab-python PEtab v2 linter.

NN model via NN model YAML format

Hybridization table

  • targetIds are not also targetIds in the condition table
  • targetIds are not also inputIds or parameterIds in the array data files
  • NN inputs are not in targetValues
  • NN outputs are not targetIds
  • targetValues contain valid PEtab math expressions
  • pre-initialization--hybridized NNs are evaluable (i.e. only depend on things that are available before simulation)

Condition table

  • targetIds are not also inputIds or parameterIds in the array data files
  • NN inputs are not in targetValues
  • NN outputs are not targetIds

Array data files

  • any input in the file must have an array defined for all initial PEtab conditions

Parameter table

  • a parameter with nominalValue:=array must be defined by an array
  • all parameters of the NN are explicitly specified to be estimated or fixed
  • if posterior then all ML parameters require one of the following
    • a proper prior
    • finite bounds

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.