`petab_math_str` writes `nan` and `zoo`, which are not valid PEtab
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 72/100
Direzione di ricerca
Inizia dal modulo petab.v1.math, dove petab_math_str converte le espressioni sympy in stringhe PEtab, e riproduci le tre chiamate nell'issue per vedere l'output con nan e zoo*x. Verifica come sympify_petab rifiuta già il nome riservato nan, così che il nuovo controllo segua lo stesso schema. È completato quando petab_math_str solleva un ValueError per le espressioni che contengono nan o zoo, e un test copre entrambi i casi.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
NaN is not representable in PEtab (v2 spec: NUMERIC excludes NaN; math expressions: "neither is NaN (not-a-number)" supported; nan is a reserved keyword, "undefined in PEtab"). The same holds for sympy's complex infinity zoo.
Nevertheless, petab_math_str prints them, producing files that cannot be read:
import sympy as sp
from petab.v1.math import petab_math_str, sympify_petab
petab_math_str(sp.nan) # 'nan'
petab_math_str(sympify_petab("x / 0")) # 'zoo*x'
sympify_petab("nan") # ValueError: Use of reserved name 'nan'
petab_math_str should raise a ValueError for expressions containing nan or zoo, instead of silently writing invalid PEtab.
🤖 Drafted with AI assistance.
- Lingua principale
- Python
- Stelle
- 18
- Fork
- 9
- Merge medio
- 1g 8h
- PR unite (30g)
- 21
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di PEtab-dev/libpetab-python
-
`petab1to2`: random column name for pre-existing `experimentId` columnForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
PEtab-dev/libpetab-python#568 ·
I maintainer di solito rispondono entro 1 giorno
-
`petab_files_1to2` keeps a stale model file when converting into a non-empty output directoryAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 73/100
PEtab-dev/libpetab-python#566 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
PEtab-dev/libpetab-python#525 ·
I maintainer di solito rispondono entro 1 giorno
-
Deprecation warnings with pandas 3.1 (`inplace`, `read_csv(float_precision=...)`)Forse già presa @dweindl l’ha presa 3 giorni fa. Aperta
PEtab-dev/libpetab-python#570 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
`petablint` fails for PEtab v2 problems with only warningsForse già presa @dweindl l’ha presa 4 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 35/100
PEtab-dev/libpetab-python#567 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di PEtab-dev/libpetab-python
Issue simili
-
The changelog workflow excludes otelbot but not dependabot, so every Dependabot pull request fails the changelog checkForse già presa @ocelotl l’ha presa oggi. Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 82/100
open-telemetry/opentelemetry-python#5825 ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
cuga-project/cuga-agent#1020 ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 70/100
microsoft/playwright#43640 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
stac-utils/stac-fastapi-elasticsearch-opensearch#965 ·
I maintainer di solito rispondono entro 2 giorni
-
from-review-extraction library privacy python severity:suggested
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
LearningCircuit/local-deep-research#7951 ·
I maintainer di solito rispondono entro 2 giorni