`petablint` fails for PEtab v2 problems with only warnings
Maintainer antworten meist innerhalb von 1 Tag
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 35/100
Rechercherichtung
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.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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)
- Vorherrschende Sprache
- Python
- Sterne
- 18
- Forks
- 9
- Ø Merge
- 1 T. 8 Std.
- Gemergte PRs (30 T.)
- 21
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus PEtab-dev/libpetab-python
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
PEtab-dev/libpetab-python#568 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 73/100
PEtab-dev/libpetab-python#566 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
PEtab-dev/libpetab-python#549 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
PEtab-dev/libpetab-python#525 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Deprecation warnings with pandas 3.1 (`inplace`, `read_csv(float_precision=...)`)Evtl. vergeben @dweindl hat das vor 1 Tag übernommen. Offen
PEtab-dev/libpetab-python#570 · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in PEtab-dev/libpetab-python
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
Maintainer antworten meist innerhalb von 1 Tag
-
enhancement good first issue
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
iceboundrock/AutoForge#304 ·
Maintainer antworten meist innerhalb von 1 Tag
-
enhancement
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
fossasia/visdom-dev#77 ·
Maintainer antworten meist innerhalb von 4 Tagen
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
aws-cloudformation/cfn-lint#4890 ·
Maintainer antworten meist innerhalb von 2 Tagen