-
Notifications
You must be signed in to change notification settings - Fork 64
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
Initial script for generating images using a python script instead of… #1184
Initial script for generating images using a python script instead of… #1184
Conversation
Co-authored-by: Andrew Hankinson <andrew.hankinson@gmail.com>
Co-authored-by: Andrew Hankinson <andrew.hankinson@gmail.com>
Co-authored-by: Andrew Hankinson <andrew.hankinson@gmail.com>
Co-authored-by: Andrew Hankinson <andrew.hankinson@gmail.com>
Co-authored-by: Andrew Hankinson <andrew.hankinson@gmail.com>
Co-authored-by: Andrew Hankinson <andrew.hankinson@gmail.com>
Co-authored-by: Andrew Hankinson <andrew.hankinson@gmail.com>
Co-authored-by: Andrew Hankinson <andrew.hankinson@gmail.com>
Co-authored-by: Andrew Hankinson <andrew.hankinson@gmail.com>
* Since having local options is currently not used we can remove the lxml dependency for now * The script will also fail if and example cannot be loaded or if the resulting SVG is empty
@lpugin looks like there's a merge conflict now. Could you please resolve this? |
…verovio-script # Conflicts: # BUILD_COMMANDLINE.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @lpugin. Tested locally and works great. Some minor suggestions below.
Co-authored-by: Stefan Münnich <stefan.muennich@unibas.ch>
Co-authored-by: Stefan Münnich <stefan.muennich@unibas.ch>
Co-authored-by: Andrew Hankinson <andrew.hankinson@gmail.com>
Co-authored-by: Andrew Hankinson <andrew.hankinson@gmail.com>
Co-authored-by: Andrew Hankinson <andrew.hankinson@gmail.com>
Co-authored-by: Andrew Hankinson <andrew.hankinson@gmail.com>
@musicEnfanthen is this also tested in the docker-mei container? |
@bwbohl I was able to execute the target from within docker-mei, yes. |
@ahankinson Are your requested changes addressed? |
Many thanks for this I think it's much better to have a script for generating the images inside this repo and should even replace the other ant target and node.js method. |
This PR adds a Python script offering an alternative to the node script for generating images with Verovio outside Docker.
The Python scripts dependencies are standard ones plus:
Both can be installed with
pip
The script can be run through a new
ant
a new targetgenerate-images-py
(guidelines should have been built beforehand):