Skip to content

Update configure.py#1977

Closed
behdad-bahadivand-chegini wants to merge 1 commit into
cuthbertLab:masterfrom
behdad-bahadivand-chegini:behdad-bahadivand-chegini-patch-1
Closed

Update configure.py#1977
behdad-bahadivand-chegini wants to merge 1 commit into
cuthbertLab:masterfrom
behdad-bahadivand-chegini:behdad-bahadivand-chegini-patch-1

Conversation

@behdad-bahadivand-chegini

Copy link
Copy Markdown

fix(configure): also set musescoreDirectPNGPath when MuseScore is chosen

The configuration assistant (SelectMusicXMLReader._performAction) now additionally writes the correct musescoreDirectPNGPath when the user selects MuseScore as the MusicXML reader. Previously only musicxmlPath was set, leaving any stale or broken value for the headless PNG path untouched. This caused SubConverterException when calling show() in Jupyter notebooks.

A helper method _getMuseScorePNGPathFromApp resolves the actual command‑line executable on macOS by looking inside the .app bundle (Contents/MacOS/mscore with fallback to MuseScore). On Windows the discovered .exe path is used directly.

fix(configure): also set musescoreDirectPNGPath when MuseScore is chosen

The configuration assistant (SelectMusicXMLReader._performAction) now
additionally writes the correct `musescoreDirectPNGPath` when the user
selects MuseScore as the MusicXML reader.  Previously only `musicxmlPath`
was set, leaving any stale or broken value for the headless PNG path
untouched.  This caused `SubConverterException` when calling `show()` in
Jupyter notebooks.

A helper method `_getMuseScorePNGPathFromApp` resolves the actual
command‑line executable on macOS by looking inside the `.app` bundle
(`Contents/MacOS/mscore` with fallback to `MuseScore`).  On Windows
the discovered `.exe` path is used directly.
@mscuthbert

Copy link
Copy Markdown
Member

Supervise your agent to not change docs it's not editing. then when your changes are focused on the issue please reopen.

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