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

FEATURE: Add Multi Modal Capabilities to Flowise #1419

Merged
merged 76 commits into from
Feb 27, 2024

Conversation

vinodkiran
Copy link
Contributor

No description provided.

# Conflicts:
#	packages/server/src/database/migrations/mysql/index.ts
#	packages/server/src/database/migrations/postgres/index.ts
#	packages/server/src/database/migrations/sqlite/index.ts
# Conflicts:
#	packages/server/src/index.ts
@HenryHengZJ
Copy link
Contributor

Update after touchup:
image

image

@HenryHengZJ
Copy link
Contributor

related chat embedded PR

@0xi4o 0xi4o marked this pull request as ready for review February 22, 2024 10:45
@0xi4o
Copy link
Contributor

0xi4o commented Feb 22, 2024

@HenryHengZJ @chungyau97 Issues have all been fixed. We can review and merge this. I think everything's good to go on @vinodkiran's end too.

@chungyau97
Copy link
Contributor

@0xi4o, thanks for the fix for invalid characters in file 2024-02-21 20_43_24-Elon Musk - Elon Musk.pdf — Mozilla Firefox.png
image

@chungyau97
Copy link
Contributor

Another error where importing flows then turn on Speech To Text will remove file and mic logo.
image

Steps to reproduce error:

  1. Click Add New
  2. Import MultiModal chatflow
  3. Save chatflow
  4. Turn on Speech To Text

@HenryHengZJ
Copy link
Contributor

HenryHengZJ commented Feb 24, 2024

Another issue:

1.) Open OpenAI Whisper, put in credential:
image

2.) switch to assembly ai, you can see the openai credential there:
image

I think that's because of the credentialNames in the useEffect, I've removed that as it caused infinite loop, but you were trying to put it there to prevent this scenario right

@HenryHengZJ
Copy link
Contributor

Another error where importing flows then turn on Speech To Text will remove file and mic logo. image

Steps to reproduce error:

  1. Click Add New
  2. Import MultiModal chatflow
  3. Save chatflow
  4. Turn on Speech To Text

solved

@HenryHengZJ
Copy link
Contributor

Another issue:

1.) Open OpenAI Whisper, put in credential: image

2.) switch to assembly ai, you can see the openai credential there: image

I think that's because of the credentialNames in the useEffect, I've removed that as it caused infinite loop, but you were trying to put it there to prevent this scenario right

solved

</div>
{inputParam.type === 'credential' && (
<CredentialInputHandler
key={speechToText[selectedProvider]?.credentialId}
Copy link
Contributor

Choose a reason for hiding this comment

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

@0xi4o this is where you can specify a key so that a new element is rendered, hence you dont need to put credentialNames in the useEffect

@ocflowiseai ocflowiseai self-requested a review February 26, 2024 13:13
}
})

this.app.get('/api/v1/get-upload-path', async (req: Request, res: Response) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Reminder to backport this route to the new way modularity express router PR - #1806

@HenryHengZJ HenryHengZJ merged commit a134ea8 into FlowiseAI:main Feb 27, 2024
2 checks passed
@HermesMacedo
Copy link

Could there be an option to configure another audio or image model (selfhosted)?

@nitromir
Copy link

nitromir commented Mar 6, 2024

Awesome feature. It would be great to add it to the chat embed as well.

JohnBQuinn pushed a commit to JohnBQuinn/Flowise that referenced this pull request Jun 7, 2024
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.

None yet

9 participants