-
- Notifications
You must be signed in to change notification settings - Fork 723
feat(bing): add image recognition functionality #452
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
Conversation
b384df5 to b403b30 Compare b403b30 to ef0f26d Compare | Hey @matibm, |
| Hey @matibm, In the sendMessage function in the Chat.vue file and I did no other modification in the API server code, except for what is already on my main branch. I'll have to do some more debugging, but a fix will probably come later this week. |
| HI! |
| Hi @vaibhavard, |
| No, without jail break it works great for me .. |
| Hey @matibm and @vaibhavard , |
| Tried and Tested.✅.
|
| Edit: I am now facing a minor bug with image recognition / searching: The initial word after API OUTPUT Gets cutoff: |
| Hey @vaibhavard, |
| Hi @Richard-Weiss If I send several messages, I get this error, approximately on message number 10 or 15 when sending images. If I don't send an image, I don't get the error. (the messages are not correlative) the message comes from this part of the codeany suggestions? thanks |
| Hey @matibm, |
| I realized that right now it doesn't work even on the official bing chat page when you upload an image. It's probably an internal problem, and it's going to work again eventually. |
| Yeah, I also tried it first and had no issue, but after some retries it seems to have some issues and it retries it a bunch of times in vanilla Bing. |
| any updates? |
| @veigamann It's ready to be merged, just waiting for a review and merge. |
| awesome, thanks! hope it gets merged soon 😉 |
using a fork of node-chatgpt-api as [this PR](waylaidwanderer/node-chatgpt-api#481) hasnt been merged yet. will revert to the main package as soon as this gets merged. this fork also implements the feat in [this PR](waylaidwanderer/node-chatgpt-api#452) but it's not yet implemented
using a fork of node-chatgpt-api as [this PR](waylaidwanderer/node-chatgpt-api#481) hasnt been merged yet. will revert to the main package as soon as this gets merged. this fork also implements the feat in [this PR](waylaidwanderer/node-chatgpt-api#452) but it's not yet implemented
using a fork of node-chatgpt-api as [this PR](waylaidwanderer/node-chatgpt-api#481) hasnt been merged yet. will revert to the main package as soon as this gets merged. this fork also implements the feat in [this PR](waylaidwanderer/node-chatgpt-api#452) but it's not yet implemented





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.