Skip to content

Mac compat#82

Open
cparish312 wants to merge 2 commits into
apirrone:devfrom
cparish312:mac_compat
Open

Mac compat#82
cparish312 wants to merge 2 commits into
apirrone:devfrom
cparish312:mac_compat

Conversation

@cparish312
Copy link
Copy Markdown

@cparish312 cparish312 commented Apr 23, 2024

Allows for basic Mac compatibility through:

  1. Reorganizing the background object to prevent serializing complex objects into the process_images workers (by initializing the objects within the run function)
  2. Commenting out print("QUEUE SIZE", self.images_queue.qsize()) as multiprocessing queue.qsize() is not implemented for Mac
  3. When on Mac utils.get_active_window() always returns None. I couldn't find a simple Mac python replacement. The best lead involved calling AppleScript through subprocess

Mac config:
Chip: Apple M3 Max
macOS: 14.1.1

@apirrone
Copy link
Copy Markdown
Owner

Thanks for your contribution !

As soon as I can get my hands on a Mac, I will test and review.

Can you include your Mac's config in the PR text ? (CPU and OS version mainly)

@hunnywar
Copy link
Copy Markdown

hunnywar commented Feb 12, 2025

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants