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

Performance improvements in the custom widget #6

Open
oldtechaa opened this issue Mar 25, 2016 · 3 comments
Open

Performance improvements in the custom widget #6

oldtechaa opened this issue Mar 25, 2016 · 3 comments

Comments

@oldtechaa
Copy link
Owner

We've been having some trouble with performance in the custom widget, so we need to continue minimizing calls to expose(). For example, when a note is removed, the button handler calls expose(). That needs to be removed by clearing only the affected rectangle and redrawing the grid around it. See Cairo documentation for more details on how to do that.

@oldtechaa oldtechaa added this to the Milestone 9 - Add more features. milestone Mar 25, 2016
@oldtechaa oldtechaa modified the milestones: Milestone 7 - Add capability for more complex files., Milestone 9 - Add more features. Mar 28, 2016
@oldtechaa oldtechaa added the major label Apr 5, 2016
@oldtechaa oldtechaa modified the milestones: Milestone 7 - Add capability for more complex files., Milestone 10 - Clean things up. May 5, 2016
@oldtechaa
Copy link
Owner Author

This issue will no longer be a problem as we switch to Gtk3 #26 with its clip regions by default and we have switched to different methods of drawing on the widget. This can be closed.

@oldtechaa oldtechaa reopened this Sep 5, 2016
@oldtechaa
Copy link
Owner Author

oldtechaa commented Mar 31, 2017

Originally, this issue was reopened because we would not switch to Gtk3. We restarted work on Gtk3 and have completed the switch, eliminating most of the need for this issue. However, there are still performance improvements that can be made, such as by switching to a $notes{"$x;$y"}[] HoA with constants for the list indices.

@oldtechaa
Copy link
Owner Author

Moved to Milestone 9 because this is not urgently needed and may increase code complexity too much to be worthwhile for the small memory gain.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant