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

Include pyobjc in requirements.txt #3580

Closed
wants to merge 4 commits into from

Conversation

dayofthedave
Copy link

@dayofthedave dayofthedave commented Apr 30, 2023

Background

When I'm setting up Auto-GPT (on MacOS using PyCharm), I get this error:

ModuleNotFoundError: No module named 'AppKit'

This issue is resolved by installing pyobjc.

Changes

Add pyobjc to requirements.txt.

Documentation

Simple change, no documentation necessary.

Test Plan

Built project before installing pyobjc: failed. Built project after installing pyobjc: success.

PR Quality Checklist

  • My pull request is atomic and focuses on a single change.
  • I have thoroughly tested my changes with multiple different prompts.
  • I have considered potential risks and mitigations for my changes.
  • I have documented my changes clearly and comprehensively.
  • I have not snuck in any "extra" small tweaks changes

@vercel
Copy link

vercel bot commented Apr 30, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
docs ⬜️ Ignored (Inspect) May 13, 2023 5:03pm

@k-boikov k-boikov added one-liner platform dependent When something works but not on all platforms Mac OS labels Apr 30, 2023
@p-i-
Copy link
Contributor

p-i- commented May 5, 2023

This is a mass message from the AutoGPT core team.
Our apologies for the ongoing delay in processing PRs.
This is because we are re-architecting the AutoGPT core!

For more details (and for infor on joining our Discord), please refer to:
https://github.com/Significant-Gravitas/Auto-GPT/wiki/Architecting

@vercel vercel bot temporarily deployed to Preview May 7, 2023 02:43 Inactive
@vercel vercel bot temporarily deployed to Preview May 12, 2023 14:34 Inactive
Copy link
Contributor

@lc0rp lc0rp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pyobjc only supports macOS. Does it ignore or fail gracefully if installed on another platform?

Since this is platform-specific; I suggest adding sys_platform == 'darwin' to make it conditional

pyobjc; sys_platform == 'darwin'

@lc0rp lc0rp added this to the v0.3.2 Release milestone May 16, 2023
@lc0rp lc0rp self-assigned this May 16, 2023
@Boostrix
Copy link
Contributor

any updates on this by people on Mac OS ?

@waynehamadi
Copy link
Contributor

I have asked on our discord if anyone had this issue in the past, I can't reproduce it and I use mac.
This is the discord thread where I asked
https://discord.com/channels/1092243196446249134/1092243197117337675/1108046954111443005

join discord: https://discord.gg/autogpt

@waynehamadi
Copy link
Contributor

@dayofthedave we're having trouble finding people that had the issue on their mac.
Let's do something: we close it and we will reopen once we find someone that has the issue, ok ?

@dayofthedave
Copy link
Author

Sure!

@dayofthedave dayofthedave deleted the pyobjc branch June 18, 2023 03:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mac OS one-liner platform dependent When something works but not on all platforms size/xs
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

6 participants