typing: `Application.step()`
Los mantenedores suelen responder en 2 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 58/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- python
- Área
- developer-experience
Línea de trabajo
Comienza por los puntos de entrada Application.step() y _step() descritos en el issue, e inspecciona todas las rutas de retorno para determinar si alguno de los dos métodos puede producir None. Alinea la anotación de retorno y el contrato de ejecución con ese comportamiento y, a continuación, verifica que desempaquetar el resultado ya no produzca las advertencias de comprobación de tipos indicadas.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Currently, Application.step() gives a bunch of squiggly red lines when trying to unpack the return value.
This is because .step() is annotated as follow, potentially returning None, which can't be unpacked
def step(self, inputs: Optional[Dict[str, Any]] = None) -> Optional[Tuple[Action, dict, State]]:
# ...
return self._step(...)
def _step(
self, inputs: Optional[Dict[str, Any]], _run_hooks: bool = True
) -> Optional[Tuple[Action, dict, State]]:
# ...
return next_action, result, new_state
(the return line of ._step() also has red squiggly lines)
Does ._step() actually ever return None ? If yes, would it make sense to have it return
(None, None, None) instead?
Having two potentially return values (the tuple or None) can be annoying to deal with if they have different "shapes". I'm guessing that directly unpacking the return value is common.
- Lenguaje dominante
- Python
- Estrellas
- 2.6k
- Forks
- 203
- Merge medio
- 3 d 10 h
- PR fusionados (30 d)
- 6
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de apache/burr
-
status/needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
apache/burr#809 · 2 comentarios ·
Los mantenedores suelen responder en 2 días
-
Streaming Event type, type hint, should support union typePosiblemente ocupada @mvanhorn la tomó hace 184 días. Abiertoarea/streaming area/typing good first issue help wanted kind/bug priority/high
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
apache/burr#607 · 12 comentarios ·
Los mantenedores suelen responder en 2 días
-
Add more instructions for applying ORM migrations for S3 deploymentPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertoarea/storage area/ui help wanted kind/documentation priority/low
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
apache/burr#524 · 11 comentarios ·
Los mantenedores suelen responder en 2 días
-
Add documentation on how to run the Burr UI serverQuizá libre de nuevo @iamSerafinnn la tomó hace 123 días y no hay ningún pull request abierto. Abiertoarea/ui good first issue kind/documentation priority/low
Dificultad 2/5 Medio día Aptitud para principiantes 68/100
apache/burr#272 · 5 comentarios ·
Los mantenedores suelen responder en 2 días
-
status/needs-triage
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
Los mantenedores suelen responder en 2 días
Todos los issues de apache/burr
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
MystenLabs/MemWal#1163 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
infertopics leaves new nodes without a topic when untopiced neighbours outnumber topiced onesPosiblemente ocupada @moneebullah25 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
ClanGenOfficial/clangen#6254 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
FinanceFlash/unvibecode#218 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día