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

Feat: Added functionality in chatgpt to upload files and choose the chat with chat_id #81

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

Anandesh-Sharma
Copy link

# Example:
client = ChatGPTClient(credential_check=False, 
                       headless=True, 
                       incognito=False, 
                       skip_login=True,
                       user_data_dir="/Users/hash/junk/talkingheads_profile")

client.interact(chat_id='676e6e30-f4e4-8007-85b0-f1f9a132dfb8',
                input_file='/Users/hash/Desktop/abc.png',
                prompt="what do you see in this image, use your own brain and only return the json with objects with the confidence. Try to be precise as much as possible.")
                

@Anandesh-Sharma
Copy link
Author

Anandesh-Sharma commented Dec 30, 2024

Please ignore, some testing is their for SSO. Later on I might add the feature of SSO too. But it has too much complexity.

This is a rough draft.

@ugorsahin
Copy link
Owner

Hi, thank you for the PR! It includes some great ideas, though it might be better to split them up—focusing on a few key changes here and moving the others into a separate PR.

Here’s what I think makes sense to include in this PR:

  • Updating Poetry
  • Adding venv to .gitignore

For the following, I have some suggestions:

  • File upload: This feature is already implemented in Google Bard and Copilot. I’ve been working on a separate fork to unify these approaches, and I’ll push it to the remote soon. Feel free to contribute to that effort for ChatGPT if you’re interested!

For this, I’m uncertain:

  • SSO login: I’m hesitant about implementing this as it poses a higher risk of breaking due to factors like different UIs provided to users (e.g., for A/B testing). A potential option could be to have a community submodule for such implementations, but I’m unsure about it for now.

Lastly, could you clarify the purpose of the server submodule? It seems like it’s intended as an example

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.

2 participants