Skip to content

Version 1.7.0#466

Merged
CarlosCancino-Chacon merged 110 commits intomainfrom
develop
Sep 18, 2025
Merged

Version 1.7.0#466
CarlosCancino-Chacon merged 110 commits intomainfrom
develop

Conversation

@CarlosCancino-Chacon
Copy link
Member

@CarlosCancino-Chacon CarlosCancino-Chacon commented Sep 2, 2025

PR for version 1.7.0

This version introduces the following changes:

New Features

Bug fixes

Other

Todo

  • Update Release notes
  • Set new version

leleogere and others added 30 commits January 28, 2025 10:29
This PR addresses potential errors in tuplets by raising a warning, it does not propose a solution or re-assignment but rather notifies the user about potentially problematic scores.
… values in order to accommodate large scores
…et_div_field_to_int64

This pull request addresses an issue with limitations of using int32 for the `onset_div` field of a note array generated from a long and complex pieces (e.g., a piano concerto, etc.)
This pull requests fixes a bug causing negative duration in ticks for performed notes.
This pull request fixes adjusting the `sound_off` of performed notes when there are tempo changes.
mgrachten
mgrachten previously approved these changes Sep 6, 2025
Copy link
Collaborator

@mgrachten mgrachten left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to go

CarlosCancino-Chacon and others added 2 commits September 8, 2025 14:56
Follow MusicXML guidelines to parse `mxl` compressed files
@ivnsp
Copy link
Collaborator

ivnsp commented Sep 10, 2025

I believe #413, #417, #418, #422, #427 have already been merged into main with the previous release 1.6.0 so they should be removed from the release notes for this one.

Copy link
Collaborator

@ivnsp ivnsp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me and will accept after the outstanding issues are resolved

@mgrachten mgrachten self-requested a review September 12, 2025 09:56
mgrachten
mgrachten previously approved these changes Sep 12, 2025
Copy link
Collaborator

@mgrachten mgrachten left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assuming Ivan's review approval is pending his requested update, I'll approve.

sildater and others added 2 commits September 12, 2025 12:22
@CarlosCancino-Chacon
Copy link
Member Author

Assuming Ivan's review approval is pending his requested update, I'll approve.

Thanks @mgrachten! I updated the CHANGES.md and updated things according to Ivan's request (now in PR #471). I created a new PR because I updated/unified the Markdown formatting of the CHANGES.md file. Once that PR is merged, we are good to go.

@ivnsp ivnsp self-requested a review September 16, 2025 14:54
Copy link
Collaborator

@ivnsp ivnsp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Considering the latest merge addressed all outstanding issues I think this is ready for release.

@CarlosCancino-Chacon CarlosCancino-Chacon merged commit a9a5689 into main Sep 18, 2025
4 checks passed
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.

9 participants