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
Timeline optimizations #5730
Timeline optimizations #5730
Conversation
Great! What about the inability to cooperate with dark mode? |
mscore/timeline.cpp
Outdated
return; | ||
} | ||
|
||
const bool layoutAll = layoutChanged && (state.startTick().negative() || state.endTick().negative()); |
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.
See https://travis-ci.org/musescore/MuseScore/jobs/652005292#L2630-L2636, "class Ms::Fraction
has no member named negative
"
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.
Looks like I forgot to commit some changes, as always. Will update PR soon.
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.
Any update on this?
This pull request doesn't address that, it is solely about performance of using Timeline. |
5b6e993
to
700003a
Compare
Partial redrawing of meta information rows is not implemented here. Also avoid rebuilding timeline grid on opening a score if timeline is invisible.
… parts with text/hbo/vbox
As full timeline refactoring (#4542) has not been finished as for this moment, this PR tries to fix the most prominent performance issues of the old Timeline implementation. In general, the following optimizations are implemented:
Although Timeline performance is still not ideal with this PR and could be optimized further, this PR should still hopefully make Timeline more usable for people working on relatively large scores. Users who don't use Timeline should also benefit from this PR due to the last optimization for invisible Timeline.