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

fixup rendering example and resizing #10

Merged
merged 1 commit into from
Aug 30, 2017
Merged

fixup rendering example and resizing #10

merged 1 commit into from
Aug 30, 2017

Conversation

wjwwood
Copy link
Member

@wjwwood wjwwood commented Aug 30, 2017

I also removed the timer doing a periodic render and added a renderLater to the resize event, since I noticed that the window would sometimes not get refreshed after a resize, leaving blank bars and stuff.

Fixes #2.

The rendering is pretty laggy too, at least on my work desktop. It's not a slouch either, GTX 1070 and 12-core i7. Not sure what that's about.

@wjwwood wjwwood self-assigned this Aug 30, 2017
@wjwwood wjwwood added the bug Something isn't working label Aug 30, 2017
@wjwwood
Copy link
Member Author

wjwwood commented Aug 30, 2017

I also refactored the rendering example to just load the ogre window and not have tabs. I had the tabs originally to play with some Qt behaviors, it also delayed loading of the ogre window until I changed tabs which was nice in a few instances, but we don't need that now.

@codebot
Copy link
Member

codebot commented Aug 30, 2017

maybe it's time to buy a GTX 1080 ?

@wjwwood
Copy link
Member Author

wjwwood commented Aug 30, 2017

... Ti, SLI ... and a gaming monitor.

@wjwwood wjwwood added the in review Waiting for review (Kanban column) label Aug 30, 2017
@greimela
Copy link

The rendering is pretty laggy too, at least on my work desktop.

I think the window currently renders just one frame per second, not caring about the hardware.

@wjwwood
Copy link
Member Author

wjwwood commented Aug 30, 2017

I think the window currently renders just one frame per second, not caring about the hardware.

Perhaps for some other reason that is still the case, but I did remove the QTimer in this pr that fired the render signal periodically once per second.

@wjwwood wjwwood merged commit d8723e5 into ros2 Aug 30, 2017
@wjwwood wjwwood deleted the fix_resizing branch August 30, 2017 21:25
@wjwwood wjwwood removed the in review Waiting for review (Kanban column) label Aug 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants