-
-
Notifications
You must be signed in to change notification settings - Fork 12
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(bing): add image recognition functionality (#452) #11
base: main
Are you sure you want to change the base?
Conversation
@danny-avila |
I'm looking to diverge from upstream since it seems the maintainer has no interest in maintaining it. The goal here is not simply to have my own fork but to ensure the community can keep maintaining the original project
There is "no need", but as this repo is publishing to npm and already at 600 weekly downloads for this week, I would appreciate a second look/test.
Does your PR on the upstream not have waylaidwanderer/node-chatgpt-api@187085f as the latest commit? it's here |
Yeah, but I thought you just wanted to merge the two PRs into this fork, or what exactly do you want to do? |
@danny-avila I'll create a new PR only containing the relevant commits for the image recognition. I also have a feature in my fork for using Local LLMs, is this something you or other users might be interested in? |
@Richard-Weiss Please review this PR as I've forked node-gpt-api to keep this library better maintained
Add groundwork for implementing the image recognition feature.
The only missing component is setting the parameters in the message dynamically.
I would appreciate any assistance.
--Edit--
I've added receiving the necessary values from the body of the /conversation request.
The imageURL or imageBase64 are added dynamically by for example PandoraAI.
Tested with image URL and direct base64 string.