An accepted pass can be refused by the open-request checks
Ya se ha fusionado un pull request relacionado.
- #83 de @CakeRepository — fusionado
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 84/100
Línea de trabajo
Empieza por las comprobaciones de open-request en src/city.rs, alrededor de las líneas 833-848, y compáralas con el comportamiento de pase aceptado documentado en docs/city.md y probado en tests/city.rs cerca de la línea 2973. Ajusta el orden o la lógica de las comprobaciones para que un pase aceptado pueda iniciar un trayecto cubierto sin devolver un 409, manteniendo los límites de solicitudes para los trayectos que necesitan un pase. Ejecuta las pruebas relevantes de city y añade cobertura para ambos escenarios descritos en el issue.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
What happened
R33 says "With an accepted pass, a trip up to max_minutes starts at once" (docs/city.md:155). A trip like that never waits, so it shouldn't be counted as a request.
The "already has a request open" check (src/city.rs:833) and the 3-requests check (src/city.rs:839) run before the pass is looked up at src/city.rs:848. So a covered trip can get a 409.
I expected it to start. The R33 test (tests/city.rs:2973) ends each ask before making the next.
Where
POST /api/trips, the ask_for_trip tool
How to see it again
Either of these gets a 409:
- A bot has an accepted 60-minute pass and its own longer request open at that house. It asks for a 30-minute call.
- A person's other bots have 3 requests waiting at the house. Their bot with the pass asks for a 30-minute call.
Found by reading the code at commit 2369ff5. Cargo wasn't available, so this hasn't been run.
- Lenguaje dominante
- Rust
- Estrellas
- 2
- Forks
- 0
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- 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 CakeRepository/agentshangout
-
say: name hint should only fire on @tags, not bare words or text inside linksPosiblemente ocupada @morganparadis la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
CakeRepository/agentshangout#501 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
CakeRepository/agentshangout#487 · 2 comentarios ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
CakeRepository/agentshangout#486 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
CakeRepository/agentshangout#460 · 1 comentario ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
CakeRepository/agentshangout#456 · 1 comentario ·
Todos los issues de CakeRepository/agentshangout
Issues similares
-
parity priority: low ui
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
storytold/vectorcraft#5660 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 79/100
qingjian-team/qingjian#901 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
longbridge/gpui-kit#3795 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 2 días