Why are exceptions in summary method ignored?
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
Línea de trabajo
Start in cortexutils/analyzer.py at lines 96-99 and trace how summary creation handles exceptions. Check the behavior for analyzers without a summary function before deciding the intended distinction. Done means the issue's exception-handling behavior is defined and covered by the relevant analyzer tests, if present.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
I see that exceptions thrown during summary creation are currently ignored:
https://github.com/TheHive-Project/cortexutils/blob/3978d779daf2f2f21aaeea7f84497bce80b839dd/cortexutils/analyzer.py#L96-L99
Why is that the case? This is quite annoying during analyzer development and I cannot think of a valid reason for this.
Is this maybe for some backwards compatibility to analyzers without a summary function? If so, can't this be made more specific?
- Lenguaje dominante
- Python
- Estrellas
- 8
- Forks
- 18
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: 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 TheHive-Project/cortexutils
-
Add typing to Worker modulePosiblemente ocupada @JuanTecedor la tomó hace 114 días. Abiertoenhancement
TheHive-Project/cortexutils#37 · 1 reacción · 2 asignados ·
-
Enroll cortexutils to codecovQuizá libre de nuevo @Kamforka la tomó hace 119 días y no hay ningún pull request abierto. Abiertoenhancement
TheHive-Project/cortexutils#36 · 1 asignado ·
-
Add or rework integration testsPosiblemente ocupada @Kamforka la tomó hace 124 días. Abiertoenhancement
TheHive-Project/cortexutils#34 · 1 asignado ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
Todos los issues de TheHive-Project/cortexutils
Issues similares
-
bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
openwisp/openwisp-utils#777 ·
-
Boolean statistics (Covered, Confident, JaccardDistant) aggregate to 1.0 without float weightsAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 63/100
google-research/weatherbenchX#310 ·
Los mantenedores suelen responder en 3 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
huggingface/peft#3955 ·
Los mantenedores suelen responder en 1 día
-
author-page correction
Dificultad 2/5 1-3 horas Aptitud para principiantes 60/100
acl-org/acl-anthology#10313 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
AOSSIE-Org/Gitcord-GithubDiscordBot#147 · 1 comentario ·
Los mantenedores suelen responder en 1 día