PartReduction divides dynamic weights by len('Dynamic')

Abierto Apto para principiantes
#2,190 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
68/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
python
Área
analytics

Línea de trabajo

Comienza en la función de ponderación predeterminada de PartReduction, que divide la suma entre la longitud del nombre de clase 'Dynamic' en lugar de entre el número de targets, como sugiere el docstring. Cambia el divisor a len(targets) y luego ejecuta testPartReductionB, testPartReductionC y testPartReductionE. Esos valores esperados cambiarán, así que confirma con un maintainer antes de editarlos. Está terminado cuando las tres pruebas pasan con expectativas aprobadas por el maintainer.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

PartReduction's default weight function returns summation / len(target), where target is the class name 'Dynamic', so it divides by 7 instead of averaging as the docstring says. A measure with two dynamics then weighs twice as much as one with one. Dividing by len(targets) fixes it; expected values in testPartReductionB, C and E change. OK to update them?

AI-assisted (Claude)

Lenguaje dominante
Python
Estrellas
2.6k
Forks
459
Merge medio
9 h 7 min
PR fusionados (30 d)
115

Preparar el entorno

Abrir en Codespaces

Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de cuthbertLab/music21

Todos los issues de cuthbertLab/music21

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.