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: issue-5712 add sleep tool #5715

Merged

Conversation

whysage
Copy link
Contributor

@whysage whysage commented Jun 5, 2023

Fixes # 5712 added sleep tool

Who can review?

@vowelparrot

@@ -160,3 +160,9 @@ Below is a list of all supported tools and relevant information:
- Notes: A connection to the OpenWeatherMap API (https://api.openweathermap.org), specifically the `/data/2.5/weather` endpoint.
- Requires LLM: No
- Extra Parameters: `openweathermap_api_key` (your API key to access this endpoint)

**sleep**
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's add to langchain/agents/load_tools.py for this one!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, done.

@vowelparrot
Copy link
Contributor

Thanks for the PR! one main request - I'm also curious what you've found this useful for :)

@whysage
Copy link
Contributor Author

whysage commented Jun 5, 2023

Thanks for the PR! one main request - I'm also curious what you've found this useful for :)

If we need to start a long-running job (for example we have a custom tool with a script that runs for several minutes or hours and save results in DB), an agent needs to wait for these results just doing nothing.

Copy link
Contributor

@hwchase17 hwchase17 left a comment

Choose a reason for hiding this comment

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

looks reasonable to me - thanks!

@hwchase17 hwchase17 added the lgtm PR looks good. Use to confirm that a PR is ready for merging. label Jun 5, 2023
@whysage
Copy link
Contributor Author

whysage commented Jun 7, 2023

@hwchase17 @vowelparrot Finally all tests passed :-). Any other concerns?

@vowelparrot vowelparrot merged commit 8ef7274 into langchain-ai:master Jun 7, 2023
Undertone0809 pushed a commit to Undertone0809/langchain that referenced this pull request Jun 19, 2023
This was referenced Jun 25, 2023
@whysage whysage mentioned this pull request Sep 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm PR looks good. Use to confirm that a PR is ready for merging.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants