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

Initial commit for autogpt-twitter plugin - Twitter API Integrations for AutoGPT using Tweepy! #7

Merged
merged 7 commits into from
Apr 21, 2023

Conversation

johnseth97
Copy link
Contributor

This is the initial commit for autogpt-twitter. View the readme.md located within the autogpt-twitter plugin for installation and setup instructions!

@ntindle
Copy link
Member

ntindle commented Apr 21, 2023

Looks like we should adjust the scaffolding repo. We shouldn’t need all the project config stuff for each project. I think you should just need to replace the twitter plugin’s init.py and add the twitter.py file in the same twitter directory

setuptools
build
twine
tweepy
Copy link
Member

Choose a reason for hiding this comment

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

Update the base repo with this

@@ -0,0 +1,40 @@
# autogpt-twitter 🐣
Copy link
Member

Choose a reason for hiding this comment

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

Move this to plugins/twitter

@@ -0,0 +1,256 @@
"""Twitter API integrations using Tweepy."""
Copy link
Member

Choose a reason for hiding this comment

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

Move to plugins/twitter

@@ -0,0 +1,92 @@
"""This module contains functions for interacting with the Twitter API."""
Copy link
Member

Choose a reason for hiding this comment

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

Move to plugins/twitter

@johnseth97
Copy link
Contributor Author

johnseth97 commented Apr 21, 2023

I'm a bit confused, do you just want me to rename the autogpt-twitter directory to twitter, and only include the __init__.py, twitter.py, requirements.txt , and readme.md files within said directory?

And within requirements.txt, only include the requirements for the plugin functionality with noting extra?

@ntindle
Copy link
Member

ntindle commented Apr 21, 2023

Yes, the rest for code quality coverage etc should use the ones from the root of the repo. I don’t think the example plug-in package is very clear about that is it?

@johnseth97
Copy link
Contributor Author

Not really, but we're figuring it out as we go!

Don't quote me, but I believe the current example plug-in package is fairly complex with a lot of extra functionality for integrating your own LLM, it's not anywhere near as simple as a plugin for adding in commands to interface with an API.

@ntindle
Copy link
Member

ntindle commented Apr 21, 2023

That makes sense. Part of the goal of getting the twitter one in first is twitter like things are a really common request so having an example of it is really helpful

@johnseth97
Copy link
Contributor Author

Alright, what do you think of the changes so far?

Comment on lines 1 to 38
[build-system]
requires = ["hatchling"]
build-backend = "hatchling.build"

[project]
name = "autogpt_twitter"
version = "0.0.1"
authors = [
{ name="desojo", email="17620345+desojo@users.noreply.github.com" },
]
description = "AutoGPT Twitter integration using the Tweepy API."
readme = "README.md"
requires-python = ">=3.8"
classifiers = [
"Programming Language :: Python :: 3",
"License :: OSI Approved :: MIT License",
"Operating System :: OS Independent",
]
dependencies = ["abstract-singleton", "tweepy", "pandas", "auto_gpt_plugin_template"]

[project.urls]
"Homepage" = "https://github.com/desojo/autogpt-twitter"
"Bug Tracker" = "https://github.com/desojo/autogpt-twitter"

[tool.black]
line-length = 88
target-version = ['py38']
include = '\.pyi?$'
extend-exclude = ""

[tool.isort]
profile = "black"

[tool.pylint.messages_control]
disable = "C0330, C0326"

[tool.pylint.format]
max-line-length = "88"
Copy link
Member

Choose a reason for hiding this comment

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

Don't think we need this one

Comment on lines 1 to 14
black
isort
flake8
pylint
abstract-singleton
wheel
setuptools
build
twine
tweepy
pandas
auto_gpt_plugin_template
python-dotenv
pathlib
Copy link
Member

Choose a reason for hiding this comment

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

Didn't really consider this. Make a new file in the base of the repo called requirements_all.txt with the following contents:

# Auto-GPT Plugins Full dependency set
-r requirements.txt

# plugins/twitter
tweepy

And include any specific packages you need

@ntindle
Copy link
Member

ntindle commented Apr 21, 2023

The changes are really good! Just a few comments as I figure this all out, sorry!

@ntindle
Copy link
Member

ntindle commented Apr 21, 2023

Wait, sorry again for you being the test pilot, but I think the path is supposed to be src/autogpt_plugins/twitter/init.py, etc., for all of these. I forgot we moved them.

@ntindle
Copy link
Member

ntindle commented Apr 21, 2023

Looking way better, See

Wait, sorry again for you being the test pilot, but I think the path is supposed to be src/autogpt_plugins/twitter/init.py, etc., for all of these. I forgot we moved them.

They were moved to simplify the imports

Copy link
Member

@ntindle ntindle left a comment

Choose a reason for hiding this comment

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

See comment above

@BillSchumacher BillSchumacher merged commit bb3d95e into Significant-Gravitas:master Apr 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants