Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

MusicXML `<transpose>` with `<octave-change>` but no `<diatonic>` raises TypeError

Abierto Apto para principiantes
  • ¿Algo no está bien?
#2,082 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
75/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
python
Área
data

Línea de trabajo

El fallo proviene de MeasureParser.xmlTransposeToInterval en musicxml/xmlToM21.py, donde la rama de cambio de octava suma a diatonicStep incluso cuando es None. Empieza ejecutando el fragmento de reproducción del issue para confirmar el TypeError, y luego lee el código posterior a esa rama para ver cómo ya se gestiona un paso diatónico ausente. Está terminado cuando el fragmento se analiza correctamente y la parte suena una octava más grave de lo escrito, con una prueba existente o nueva que cubra la transposición de solo octava.

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

Descripción

music21 version

10.5.0

Problem summary

<diatonic> is optional in <transpose>, and the importer handles it being missing, except when there's also an <octave-change>. Then it adds the octave to a None diatonic step and raises. An octave-only <transpose> is how guitar and bass parts are written, and some exporters leave out <diatonic> when it's 0.

Steps to reproduce

from music21 import converter

xml = '''<score-partwise version="4.0">
  <part-list><score-part id="P1"><part-name>Music</part-name></score-part></part-list>
  <part id="P1"><measure number="1">
    <attributes>
      <divisions>1</divisions>
      <transpose><chromatic>0</chromatic><octave-change>-1</octave-change></transpose>
    </attributes>
    <note><pitch><step>C</step><octave>5</octave></pitch><duration>4</duration><type>whole</type></note>
  </measure></part>
</score-partwise>'''

converter.parse(xml, format='musicxml')

Expected vs. actual behavior

Expected a part that sounds an octave lower than written. Actual:

TypeError: unsupported operand type(s) for +=: 'NoneType' and 'int'

More information

In MeasureParser.xmlTransposeToInterval (musicxml/xmlToM21.py):

if mxOctaveChange is not None:
    octaveChange = int(mxOctaveChange.text) * 12
    diatonicStep += 7 * int(mxOctaveChange.text)

Only adding to diatonicStep when it isn't None should be enough, since the code after this already handles a missing diatonic step.

Lenguaje dominante
Python
Estrellas
2.6k
Forks
459
Merge medio
1 d 7 h
PR fusionados (30 d)
40

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.