Skip to content

Fill instrument abbreviations#1864

Merged
mscuthbert merged 3 commits intomasterfrom
instrument-abbrev
Feb 24, 2026
Merged

Fill instrument abbreviations#1864
mscuthbert merged 3 commits intomasterfrom
instrument-abbrev

Conversation

@mscuthbert
Copy link
Member

@mscuthbert mscuthbert commented Feb 24, 2026

(AI-Assisted -- testing if Codex can get through old maintenance tasks.)

  • fill all instrumentAbbreviation TODO placeholders in music21/instrument.py
  • use score-style abbreviations aligned with existing naming conventions
  • adjust specific abbreviations per follow-up review (Fretless Bass, Tam-Tam, Siren, and Hi-Hat)

@mscuthbert mscuthbert changed the title Fill instrument abbreviation TODOs (AI-Assisted) Fill instrument abbreviation TODOs Feb 24, 2026
@mscuthbert mscuthbert changed the title Fill instrument abbreviation TODOs Fill instrument abbreviations Feb 24, 2026
@coveralls
Copy link

coveralls commented Feb 24, 2026

Coverage Status

coverage: 93.061% (+0.003%) from 93.058%
when pulling f35f7d8 on instrument-abbrev
into 10d4948 on master.

(AI-Assisted)

- fix ModuleGather._getNamePeriod to strip only the file extension suffix
- remove setup-python pip cache settings from maincheck jobs that use uv cache
@mscuthbert mscuthbert merged commit e4eee83 into master Feb 24, 2026
8 checks passed
@mscuthbert mscuthbert deleted the instrument-abbrev branch February 24, 2026 22:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants