`petablint` fails for PEtab v2 problems with only warnings
Les mainteneurs répondent en général sous 1 jour
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 35/100
Piste de recherche
Read the petab/petablint.py exit handling and compare it with the PEtab v1 behavior described in the issue. Check how validation_issues.has_errors() is used and run petablint on a v2 problem that produces only warnings. Done means warnings are logged while the command exits successfully; errors should still produce exit code 1.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
For PEtab v2 problems, petablint exits with 1 if lint_problem returns any issues, including warnings (petab/petablint.py, if validation_issues: ... sys.exit(1)). For PEtab v1, only errors make it fail.
For example, unused conditions are allowed by the specification, and CheckUnusedConditions reports them as WARNING, but petablint still fails:
$ petablint problem.yaml; echo $?
WARNING: Conditions {'pred2', 'pred3', 'pred4', 'pred5'} are not used in the experiments table. [CheckUnusedConditions]
1
(Giordano_Nature2020 from the benchmark collection converted with petab1to2; same for Fujita_SciSignal2010.)
This makes petablint unusable for CI checks of valid v2 problems. Suggested fix: always log the issues, but exit with 1 only if validation_issues.has_errors(). If failing on warnings is wanted, it could be an opt-in flag.
🤖 Generated by an AI agent (Claude Code)
- Langage dominant
- Python
- Étoiles
- 18
- Forks
- 9
- Merge moyen
- 1 j 8 h
- PR mergées (30 j)
- 21
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de PEtab-dev/libpetab-python
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
PEtab-dev/libpetab-python#568 ·
Les mainteneurs répondent en général sous 1 jour
-
`petab_files_1to2` keeps a stale model file when converting into a non-empty output directoryOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 73/100
PEtab-dev/libpetab-python#566 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
PEtab-dev/libpetab-python#549 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
PEtab-dev/libpetab-python#525 ·
Les mainteneurs répondent en général sous 1 jour
-
Deprecation warnings with pandas 3.1 (`inplace`, `read_csv(float_precision=...)`)Peut-être pris @dweindl l’a pris il y a 1 jour. Ouverte
PEtab-dev/libpetab-python#570 · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de PEtab-dev/libpetab-python
Issues similaires
-
[Resource]: LoreOuverteresource-submission validation-passed
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
hesreallyhim/awesome-claude-code#3234 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
openminion/openminion#376 ·
Les mainteneurs répondent en général sous 1 jour
-
Type: ☹︎ Bug
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
posit-dev/great-tables#1028 ·
Les mainteneurs répondent en général sous 1 jour
-
integration: apple_tv
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
home-assistant/core#188282 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
A "since" range is worded as a career: "the past two seasons" says "regular season career (2025 on)"OuverteP4: low
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
jeffknupp/association#538 ·