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

jigsawstack[major]: Add Jigsawtack Integration #6665

Closed

Conversation

iamNarcisse
Copy link
Contributor

@iamNarcisse iamNarcisse commented Aug 30, 2024

Add JigsawStack agent tools.

Leveraging JigsawStack AI SDK, this PR adds the following agent tools:

  1. JigsawStackAIScrape - This tool enables you to scrape web content using advanced AI capabilities provided by JigsawStack.

  2. JigsawStackAISearch - This tool allows you to perform web searches and retrieve high-quality results powered by AI.

  3. JigsawStackSpeechToText - Transcribe video and audio files using the Whisper large V3 AI model.

  4. JigsawStackVOCR - This tool accurately recognizes, describes, and retrieves data from images.

  5. JigsawStackTextToSQL - This tool helps generate semantically correct SQL queries from text.

  6. JigsawStackPromptEngine - This tool generates text completions with support for structured outputs.

@dosubot dosubot bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label Aug 30, 2024
Copy link

vercel bot commented Aug 30, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
langchainjs-docs ✅ Ready (Inspect) Visit Preview Sep 27, 2024 9:07pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
langchainjs-api-refs ⬜️ Ignored (Inspect) Sep 27, 2024 9:07pm

@dosubot dosubot bot added the auto:enhancement A large net-new component, integration, or chain. Use sparingly. The largest features label Aug 30, 2024
@iamNarcisse iamNarcisse changed the title add jigsawtack integration jigsawstack[major]: Add Jigsawtack Integration Sep 4, 2024
Copy link
Collaborator

@jacoblee93 jacoblee93 left a comment

Choose a reason for hiding this comment

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

Thanks for the PR!

We'd need to write docs for this - but we've also been trying to move new partner integrations into separate repos to make management a bit easier. You could still add docs to the main langchain repo and they'd appear under:

https://js.langchain.com/v0.2/docs/integrations/platforms/

You could then manage your own releases as @jigsawstack/langchain or something similar, or could grant us access to the repo and we could push releases as @langchain/jigsawstack, whichever you prefer.

We could also move these into @langchain/community.

deno.lock Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you remove this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure will do

get lc_secrets(): { [key: string]: string } | undefined {
return {
apiKey: "JIGSAWSTACK_API_KEY",
api_key: "JIGSAWSTACK_API_KEY",
Copy link
Collaborator

Choose a reason for hiding this comment

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

You don't need the snake cased version of this

};
}

get lc_aliases(): { [key: string]: string } | undefined {
Copy link
Collaborator

Choose a reason for hiding this comment

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

You don't need this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Alright.

@jacoblee93 jacoblee93 added the question Further information is requested label Sep 4, 2024
@iamNarcisse
Copy link
Contributor Author

iamNarcisse commented Sep 6, 2024

Thanks for the PR!

We'd need to write docs for this - but we've also been trying to move new partner integrations into separate repos to make management a bit easier. You could still add docs to the main langchain repo and they'd appear under:

https://js.langchain.com/v0.2/docs/integrations/platforms/

You could then manage your own releases as @jigsawstack/langchain or something similar, or could grant us access to the repo and we could push releases as @langchain/jigsawstack, whichever you prefer.

We could also move these into @langchain/community.

Hi @jacoblee93,

I have made the requested changes. Let me know if it's all good.

On how releases would be deployed, We'd prefer releases as @langchain/jigsawstack. The docs have been prepared and will be included in a separate PR.

@jacoblee93
Copy link
Collaborator

On how releases would be deployed, We'd prefer releases as @langchain/jigsawstack. The docs have been prepared and will be included in a separate PR.

In that case, could you make a separate repo and add add me as an admin? I'll be able to cut releases from there.

@iamNarcisse
Copy link
Contributor Author

On how releases would be deployed, We'd prefer releases as @langchain/jigsawstack. The docs have been prepared and will be included in a separate PR.

In that case, could you make a separate repo and add add me as an admin? I'll be able to cut releases from there.

Alright

@iamNarcisse
Copy link
Contributor Author

@jacoblee93, made PR for the docs here #6767

Copy link
Member

@bracesproul bracesproul left a comment

Choose a reason for hiding this comment

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

@iamNarcisse if you want the code to be released as @langchain/jigsawstack you'll have to remove the libs/langchain-jigsawstack dir in this PR & create a separate repo (managed by you, but with admin perms granted to jacob/myself. Alternatively you can move the code into libs/langchain-community.

Please pig me when one of these has been done & I can take a look!

@iamNarcisse
Copy link
Contributor Author

@iamNarcisse if you want the code to be released as @langchain/jigsawstack you'll have to remove the libs/langchain-jigsawstack dir in this PR & create a separate repo (managed by you, but with admin perms granted to jacob/myself. Alternatively you can move the code into libs/langchain-community.

Please pig me when one of these has been done & I can take a look!

Hi @bracesproul, thanks for the feedback. I have removed dir libs/langchain-jigsawstack and made a separate PR for the docs, see #6767. I have added @jacoblee93 to the forked repo that contains the code.

@jacoblee93 jacoblee93 closed this Sep 27, 2024
@langchain-ai langchain-ai locked as spam and limited conversation to collaborators Oct 9, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
auto:enhancement A large net-new component, integration, or chain. Use sparingly. The largest features question Further information is requested size:XXL This PR changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants