Hacktoberfest 2026：メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

MusicXML import drops piano-family MIDI programs when the instrument name isn't recognized

オープン 初心者向け
  • 何か問題がありますか？
#2,084 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
72/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
python
領域
data

調査の方向性

バグは MeasureParser の score-part 処理にあり、musicxml/xmlToM21.py 内で、楽器名が認識されないとピアノ系の置換が midiProgram を落としてしまいます。issue の再現スクリプトを実行すると、プログラム 1 と 2 で Instrument の midiProgram が None になることが確認できます。プログラム 1 と 2 が midiProgram として 0 と 1 を保持し、プログラム 41 が引き続き 40 の Violin を返せば完了です。修正方法は、名前が認識された場合にのみ置換するか、i.midiProgram を置換先にコピーするかのいずれかを選び、既存の MusicXML パースのテストの隣に回帰テストを追加してください。

索引モデルが issue の本文から書いたものです。

説明

music21 version

10.5.0

Problem summary

When a part's <midi-program> is one music21 maps to Piano (MusicXML programs 1–6), the importer replaces that instrument with the one made from <instrument-name>. That makes sense when the name is recognized, since exporters often write piano as a default program. But when the name isn't recognized, the replacement is a plain Instrument with no midiProgram, so the program is lost. Other programs are kept.

Steps to reproduce

from music21 import converter, instrument

def score(midi_program):
    return f'''<score-partwise version="4.0">
  <part-list><score-part id="P1"><part-name>Music</part-name>
    <score-instrument id="P1-I1"><instrument-name>Doubler</instrument-name></score-instrument>
    <midi-instrument id="P1-I1"><midi-program>{midi_program}</midi-program></midi-instrument>
  </score-part></part-list>
  <part id="P1"><measure number="1">
    <attributes><divisions>1</divisions></attributes>
    <note><pitch><step>C</step><octave>4</octave></pitch><duration>4</duration><type>whole</type></note>
  </measure></part>
</score-partwise>'''

for midi_program in (41, 1, 2):
    parsed = converter.parse(score(midi_program), format='musicxml')
    print(midi_program, [(type(i).__name__, i.midiProgram)
                         for i in parsed.recurse().getElementsByClass(instrument.Instrument)])

Expected vs. actual behavior

Expected a midiProgram of 40, 0 and 1. Actual:

41 [('Violin', 40)]
1 [('Instrument', None)]
2 [('Instrument', None)]

More information

The replacement is in MeasureParser's score-part handling (musicxml/xmlToM21.py):

# 2. midiProgram is Piano (often this is encoded only as piano for convenience)
if inst_from_name.midiProgram == i.midiProgram or isinstance(i, instrument.Piano):
    i = inst_from_name

This isn't asking to remove the piano rule, only to not lose the program when the name doesn't give an instrument, for example by only replacing when the name was recognized, or by keeping i.midiProgram on the replacement.

主要言語
Python
スター
2.6k
フォーク
459
平均マージ
1日 7時間
マージ済み PR（30日）
40

環境構築

Codespaces で開く

このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

cuthbertLab/music21 のほかの issue

cuthbertLab/music21 の issue をすべて見る

似ている issue

Python の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。