Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jupyter: Lab compatibility & Display all pages, refactor MuseScore #1592

Merged
merged 10 commits into from
Jun 4, 2023

Conversation

mscuthbert
Copy link
Member

@mscuthbert mscuthbert commented Jun 4, 2023

Users of Jupyter and MuseScore will now see all pages of a score when .show() is called in Jupyter.

Also adds compatibility for Jupyter Notebook v7 beta and I believe JupyterLab.

with .show(multipageWidget=True) an interactive widget to view multiple pages appears. I thought this was going to be the default, but it turns out I much prefer just setting all pages.

Many Converter commands are now staticmethods

subConverter now consistently spelled with capital C (backwards incompatible)

museScore functions moved to converter/museScore -- now can be run separate from export!

Jupyter functionality mostly moved to ipython21 where it belongs.

@coveralls
Copy link

coveralls commented Jun 4, 2023

Coverage Status

coverage: 92.988% (-0.1%) from 93.116% when pulling 3b6a927 on file_widget into 2ca1f13 on master.

Refactor out Jupyter code to the ipython21 directory

Refactor out MuseScore code to converter/museScore

Make subConverter consistently have a capital C (backwards incompatible)
@mscuthbert mscuthbert changed the title Jupyter: Lab compatibility & Display all pages of a file Jupyter: Lab compatibility & Display all pages, refactor MuseScore Jun 4, 2023
@mscuthbert mscuthbert merged commit 58f5495 into master Jun 4, 2023
@mscuthbert mscuthbert deleted the file_widget branch June 4, 2023 02:37
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