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

rqt_image_view: Fixed image shirinking problem. #291

Closed
wants to merge 3 commits into from

Conversation

mjyc
Copy link
Contributor

@mjyc mjyc commented Nov 12, 2014

If the greatestCommonDivisor is 1, every time setAspectRatio function gets called, width & height in aspect_ratio_ decreases. As a result, rqt_image_view shrinks the image as the times goes. This is one approach to prevent such problem.

@ablasdel ablasdel changed the title Fixed image shirinking problem. rqt_image_view: Fixed image shirinking problem. Dec 1, 2014
@dirk-thomas
Copy link
Contributor

The patch does not look like the right fix for the problem.

Can you please try to modify the following two lines to round the double values to integers rather then casting them (which implies floor):

@mjyc
Copy link
Contributor Author

mjyc commented Dec 10, 2014

modified the code according to suggestion and it seems to work pretty well.

* upstream/groovy-devel:
  added check to ensure we do not fail if PyQt5 is not in sys.modules
dirk-thomas pushed a commit that referenced this pull request Dec 10, 2014
@dirk-thomas
Copy link
Contributor

Thanks, I have manually merged it in: 95f0155

mjyc pushed a commit to mjyc/rqt_common_plugins that referenced this pull request Feb 5, 2015
* upstream/groovy-devel:
  Fix version clash when PyQt5 is installed
  Remove dead nodes from memory and rebuild uri cache when nodes are restarted
  fix image shrinking problem ros-visualization#291
  rqt_reconfigure: Restore support for parameter groups
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.

2 participants