RandomTable.roll() returning None fairly freqently.
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 45/100
Línea de trabajo
Busca la implementación de RandomTable.roll() e inspecciona el bucle mostrado en el issue. Verifica el comportamiento en los límites para tiradas de 0 y para pesos acumulados exactos; después, prueba mapas aleatorios desde el capítulo 45 en adelante y confirma que las tiradas válidas ya no devuelvan None ni produzcan la advertencia.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
To see the problem in action, get to a random map from chapter 45 on-wards.
The message "WARNING: We don't know how to spawn [None]!" should show up a number of times in the log.
Currently, whenever you roll 0, or exactly the sum of a number of item weights, nothing is spawned.
For example, if the first few items are weighted at [1,3,2,3,1...], then roll() would return "None" if the rng hits 0,1,4,6,9 or 10.
I'm assuming this is unintended behavior, but you can see pretty clearly why this happens in the code:
while roll > 0 {
if roll < self.entries[index].weight {
return self.entries[index].name.clone();
}
roll -= self.entries[index].weight;
index += 1;
}
There are two easy fixes that I can see. Both effectively just looping more:
while roll >= 0 { ...
or
while index < self.entries.len() { ..
- Lenguaje dominante
- Rust
- Estrellas
- 968
- Forks
- 166
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Aún no hemos revisado los archivos de configuración de este proyecto. Empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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 amethyst/rustrogueliketutorial
-
Wrong typing in chapter 57aAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
-
[RESOLVED] Wayland IssueAbierto
Dificultad 1/5 1-3 horas Aptitud para principiantes 45/100
Todos los issues de amethyst/rustrogueliketutorial
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
linebender/parley#849 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
boxlite-ai/boxlite#1729 ·
Los mantenedores suelen responder en 1 día
-
external
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
langchain-ai/docs#6255 ·
Los mantenedores suelen responder en 1 día
-
P2 testing
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
Los mantenedores suelen responder en 1 día