Skip to content
This repository has been archived by the owner on Feb 15, 2024. It is now read-only.

Empty zoom source list [Linux] [BSPWM] #76

Closed
Flinner opened this issue Oct 17, 2022 · 2 comments
Closed

Empty zoom source list [Linux] [BSPWM] #76

Flinner opened this issue Oct 17, 2022 · 2 comments

Comments

@Flinner
Copy link

Flinner commented Oct 17, 2022

This is related to #74 but i see no error logs at all.

testing these lines manually

def update_sources(self):
"""
Update the list of Windows and Monitors from PyWinCtl
"""
self.windows = pwc.getAllWindows()
self.monitors = pwc.getAllScreens()
self.monitors_key = list(dict.keys(self.monitors))

gives

>>> import pywinctl as pwc
>>> pwc.getAllWindows()
[LinuxWindow(hWnd=<class 'Xlib.display.Window'>(0x01200003)), .....]
>>> pwc.getAllScreens()
{'eDP-1': ........}}

so the library works (?) for me, but the source list is still empty

I use v2022.09.26, copied the script directly from the repo (so it is latest version)

when i press "Refresh list of sources" nothing happens, zero logs from the script

thanks for making this tool, any help is appreciated

@Flinner
Copy link
Author

Flinner commented Oct 17, 2022

if source_type in { "monitor_capture", "window_capture",
"game_capture", "display_capture" }:

my source is of type "xshm_input", tried adding it like so:

            if source_type in { "monitor_capture", "window_capture",
                                "game_capture","xshm_input", "display_capture" }:

it is now detected! but look how "zooming" looks like :(
i have to type manually the resolution to remove the zoom

2022-10-17.13-26-39.mp4

To describe it, i would say my video is cropped not zoomed

EDIT

I followed instructions about "Edit Transform" and now it works, but i had to manually set screen size for it to work

tryptech added a commit that referenced this issue Oct 21, 2022
Should address #74 and #76 for Linux systems using X Window System/X11
@tryptech
Copy link
Owner

Thank you for finding that information, I have updated the script accordingly, but I'm not sure if monitor sizes will be reported properly. If not, you will have to continue to manually set the screen size.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants