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

Hidden treatment images #6

Open
hardfreqz opened this issue May 22, 2018 · 4 comments
Open

Hidden treatment images #6

hardfreqz opened this issue May 22, 2018 · 4 comments
Labels
question Further information is requested

Comments

@hardfreqz
Copy link

hardfreqz commented May 22, 2018

I can't see the windows of
"original_warped"
"adaptiveTreshold"
"merge"

The windows are invisible.
capture

screenshot_1

Any idea how to fix that ?

@zappybiby
Copy link
Owner

The relevant code is here: https://github.com/bethesirius/ChosunTruck/blob/889644385ce57f971ec2921f006fbb0a167e6f1e/windows/src/main.cpp#L140

We divide the width and height by a certain value so that it (in theory) works for any monitor size. Find the coordinates of where you would like to have the window display and calculate what you would need to divide the width by to move to those coordinates. I wonder if the windows are being drawn behind the game - to check, hover over the windows that you see in your second screenshot to see if they appear in front of the game.

The windows are being drawn as seen in your second screenshot so I believe the issue is with the way the windows are being moved and not an issue with the windows not being displayed.

@zappybiby
Copy link
Owner

zappybiby commented May 22, 2018

Oh my goodness. I am on my phone and thought this was for another project.

The window positions are hardcoded as I didn't expect anyone to look at this project yet. The issue is that the windows are hardcoded to work on computers with two monitors.

Change these parameters to the coordinates of your choosing:

cv2.resizeWindow("merge", 450, 500)

@hardfreqz
Copy link
Author

Not problem guy ! And Thank you for spend your time for me !
Thank you for all, i will test later.

@zappybiby
Copy link
Owner

Did that resolve the issue?

@ajchili ajchili added the question Further information is requested label May 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants