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

High Resolution, unable to mouse select roi #59

Closed
KevParris opened this issue Sep 6, 2021 · 2 comments
Closed

High Resolution, unable to mouse select roi #59

KevParris opened this issue Sep 6, 2021 · 2 comments
Labels
Milestone

Comments

@KevParris
Copy link

I have video with resolution 2592x1944 and want to select a rectangle of interest using the mouse.
The image presented by DVR-Scan for this is full-scale and much larger than my 1920x1080 physical screen.
I have not figured out a way to move the image around so the portion where I want to click/drag is visible.
If I right-click on the image title bar and choose move, the cursor up-arrow key will move the image up but Windows snaps it back down to keep the title bar visible before I can do anything else.

I realize re-scaling the image to fit the screen will affect precision/accuracy of the click/drag positioning, but I think it would be good enough

Or the image could be presented in a frame with scroll bars perhaps? And maybe with a zoom in/out feature?

@Breakthrough
Copy link
Owner

Unfortunately right now DVR-Scan just uses the OpenCV GUI for displaying the image, which makes any kind of scrolling difficult to implement. However, it should be feasible to somehow detect the user's resolution, and downscale the preview image accordingly.

As a temporary workaround, you can export a single frame of your video using ffmpeg, and use a photo editor like Paint or GIMP to figure out the exact bounding box you need. Then you can pass those values directly to --roi.

Hope this helps for now, thanks for the report!

@Breakthrough Breakthrough added this to the v1.4 milestone Oct 22, 2021
Breakthrough added a commit that referenced this issue Oct 22, 2021
@Breakthrough
Copy link
Owner

Breakthrough commented Oct 23, 2021

Completed in v1.4 branch, will be included in next release.

If you're installing via pip, install the screeninfo package to resolve this issue.

Thanks for the bug report!

@Breakthrough Breakthrough pinned this issue Oct 23, 2021
@Breakthrough Breakthrough unpinned this issue Feb 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants