Stop assuming all upperclassmen are on every packet
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 52/100
Línea de trabajo
Comienza con packet/templates/upperclassmen_totals.html en la línea 39 y packet/routes/upperclassmen.py en las líneas 51-62. Sigue cómo llegan los datos de los upperclassmen a la sección activa y, después, asegúrate de que el denominador mostrado cuente solo los paquetes que cada upperclassman puede firmar. Se considera hecho cuando los conteos activos usan num_can_sign_as_upper en lugar de num_total cuando el estado de membresía excluye los paquetes anteriores.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
This line makes the assumption that every single upperclassman is a required signature on every packet. This may not be the case, due to e.g. changes in membership state that don't influence earlier packets. This results in a confusing list of signature counts when an upperclassman is required on only some of the packets, since they are listed as $num_signed / $num_total in the active section. This aught to be $num_signed / $num_can_sign_as_upper
https://github.com/ComputerScienceHouse/packet/blob/f61cd2b5b616e3b7f35fad95b515313ea2d50bcb/packet/templates/upperclassmen_totals.html#L39
This will probably require injecting more data into the upperclassmen dict here so that it's accessible in the template.
https://github.com/ComputerScienceHouse/packet/blob/f61cd2b5b616e3b7f35fad95b515313ea2d50bcb/packet/routes/upperclassmen.py#L51-L62
- Lenguaje dominante
- Python
- Estrellas
- 25
- Forks
- 33
- Merge medio
- 21 d 1 h
- PR fusionados (30 d)
- 1
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 ComputerScienceHouse/packet
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
ComputerScienceHouse/packet#413 ·
-
Upgrade to bootstrap 5Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 38/100
ComputerScienceHouse/packet#411 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
ComputerScienceHouse/packet#408 ·
-
React Frontend?Abiertoux enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
ComputerScienceHouse/packet#404 ·
-
ux enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
ComputerScienceHouse/packet#403 ·
Todos los issues de ComputerScienceHouse/packet
Issues similares
-
correction metadata
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
acl-org/acl-anthology#10104 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
bug status/needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
prowler-cloud/prowler#12885 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Bug in GaussianTailProbabilityCalibrator: running_statistics=False still uses a windowed varianceAbiertobug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
selimfirat/pysad#107 ·
Los mantenedores suelen responder en 1 día
-
bug ci-failure high priority
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
vllm-project/vllm-omni#8194 · 1 comentario ·
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