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

Add graphics acceleration indicator to Debug widget #44

Merged
merged 2 commits into from
Jan 31, 2016

Conversation

Treer
Copy link
Collaborator

@Treer Treer commented Jan 26, 2016

Might be a bit of an overkill, since I suspect graphics are accelerated for nearly everyone, but I'd like to keep an eye on it during development.

There's now a dot on the FPS widget that's green if everything's accelerated, and blue if it's not. The Debug widget lists the value as a percentage.

@Treer
Copy link
Collaborator Author

Treer commented Jan 26, 2016

If acceleration is working in 99.9% of computers then we should remove this indicator from the FPS widget - to save people wondering what it is.

Also, the network graph suggests I'm still not doing this branch thing properly. I think it's because I forgot to pull to my computer and checkout master before creating the next branch. Let me know if it's messed up from your end.

I'll get it right next time ;)

@stefandollase
Copy link
Contributor

I will look into this now. I think it is a good idea to add it to the debug widget. The numbers in there are generally confusing, so one more won't hurt :-P

Regarding the branching: You already figured it out. Before you create a branch you need to pull and checkout master. I will try to fix it for the messed up branches by rebaseing and force pushing. This might cause problems on your end, however these problems should vanish as soon as the pull request is merged.

Treer and others added 2 commits January 31, 2016 21:08
* used only one instance of the counter class
* passed counter to the relevant parts via dependency injection
* removed counter from widget and fragment drawer since it does not really belong there
* removed dot from fps widget to reduce users confusion
@stefandollase
Copy link
Contributor

I removed the dot from the FPS widget. This will hopefully reduce the confusion. Other than that I do like the idea to include it as a debug information. This should help us to sort out the issue with the unaccelerated graphics if it is an issue for many users.

@stefandollase stefandollase changed the title Add graphics acceleration indicators to FPS and Debug widgets Add graphics acceleration indicator to Debug widget Jan 31, 2016
@stefandollase stefandollase added this to the v4.0 milestone Jan 31, 2016
stefandollase added a commit that referenced this pull request Jan 31, 2016
Add graphics acceleration indicator to Debug widget
@stefandollase stefandollase merged commit 6915055 into master Jan 31, 2016
@stefandollase stefandollase deleted the acceleration-indicator branch January 31, 2016 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants