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

Score model wrong without sending it through serializer #551

Closed
3 of 6 tasks
Danielku15 opened this issue Mar 6, 2021 · 0 comments · Fixed by #610
Closed
3 of 6 tasks

Score model wrong without sending it through serializer #551

Danielku15 opened this issue Mar 6, 2021 · 0 comments · Fixed by #610
Assignees
Labels
area-rendering Everything related to the rendering platform-java Related to the Java version of alphaTab platform-javascript Related to the JavaScript version of alphaTab platform-net Related to the .net version of alphaTab state-accepted This is a valid topic to work on. type-bug 🕷️

Comments

@Danielku15
Copy link
Member

Observed Results

When the score model is directly sent into the ScoreRenderer after loading it, it seems that various details are wrong in the overall model. Especially regarding the timing.

This requires currently the unit tests to send it once through the JsonSerializer.

Expected Results

The Score datamodel should be 100% after loading and calling finish.

Steps to Reproduce

  1. Remove the serializer part on any of the visual unit test platforms
  2. Notice how the rendering fails due to various details being wrong in the model

Further details

As already so often in the past already, the grace timing seems to be one bad player in this bug. The GraceGroup.addBeat uses the absoluteDisplayStart for the unique IDs. But when the first beat is added in Voice.finish on line 187, the timings are not yet defined.

A fix could be to just update the IDs of the grace groups after the display and playback times are defined.

Your environment

Version

  • 1.0
  • 1.1
  • 1.2
  • 1.3-alpha.109

Flavor

  • JavaScript
  • .net
@Danielku15 Danielku15 added type-bug 🕷️ area-rendering Everything related to the rendering state-accepted This is a valid topic to work on. platform-javascript Related to the JavaScript version of alphaTab platform-net Related to the .net version of alphaTab platform-java Related to the Java version of alphaTab labels Mar 6, 2021
@Danielku15 Danielku15 self-assigned this Mar 6, 2021
@Danielku15 Danielku15 added this to the [v1.3.0] Platform Improvements and Maintenance milestone Nov 4, 2021
@Danielku15 Danielku15 moved this to Done in Version 1.3 Feb 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-rendering Everything related to the rendering platform-java Related to the Java version of alphaTab platform-javascript Related to the JavaScript version of alphaTab platform-net Related to the .net version of alphaTab state-accepted This is a valid topic to work on. type-bug 🕷️
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant