-
Notifications
You must be signed in to change notification settings - Fork 19
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
Version 1.6.0 #398
base: main
Are you sure you want to change the base?
Version 1.6.0 #398
Conversation
Updated Merge Reportbelow the updated notes for this PR to include most recent changes PR for version 1.6.0This new version addresses multiple changes, bug fixes and new features: New features
Bug fixes
Other Changes
|
option to to force new segments in `add_segments`
Fixes issue #421
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.
This release PR brings several valuable features and improvements:
- Measure feature with full I/O support for MusicXML, MIDI, Match, and Note Array
- Clef feature and an updated feature map
- Stem direction processing
- Cross-voice processing in MEI
Additionally, this release includes various bug fixes and enhancements.
There might be issues with this PR in some rare cases: as a parsing error can happen when there are invisible objects that are part of some kind of "group". For example, in #419, the parser crashes because a single note of a chord is invisible. I did not test with other types of "groups" (tuplets, beamings), but I guess it might happen too, depending on how the parser works. As long as Partitura does not have a proper way of dealing with invisible objects (such as an Note that when |
…moved to pt util function for converting ticks to seconds
This is a good bug fix for a pressing issue when loading MIDI files with tempo changes as performances.
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.
The pull request seems very comprehensive. I left a small comment about using ensure_notearray
instead of using the note_arrray
method of an object, but this should be an easy fix
…ed_score. (addressing reviewer comment)
This reviewer is now the assignee therefore review is dismissed and assigned to somebody else.
Hi there! I've incorporated the suggested changes and made the necessary updates. We're now aiming for a new release on Thursday, February 27th. Please feel free to share any additional comments or ideas until then. Thanks for your great feedback! |
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.
"Hi! I reduced my second review on the new invisible object warning since the other updates were either my own or related to package metadata. Everything looks good to me—I think we're ready to release!"
PR for version 1.6.0
This new version addresses multiple changes, bug fixes and new features:
New features
Bug fixes
get_time_maps_from_alignment
fix issue 358 #360Other Changes
Todo