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

docs: Update docstring for generate function #1391

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Hassan-Memon
Copy link
Contributor

Corrected the docstring to describe the return value as 'dict: The updated state, including the LLM's generated answer' This update enhances clarity and precision in the documentation.

PR Details:

  • Contribution: Updated docstring in the generate function

Sure, here’s your request with some corrections:


Request for a Shoutout on My LinkedIn Post

Hi LangChain Team,

I hope you're doing well. I wanted to share some exciting news about my open-source contributions. Recently, I made my very first contributions to LangChain and LangGraph, with four commits merged—two for LangChain and two for LangGraph. It’s been a fulfilling experience, and I'm excited to continue contributing to these amazing projects.

I made a LinkedIn post to celebrate this milestone, which you can check out here. It would mean a lot to me if the LangChain team could give it a shoutout or some recognition, either by commenting on the post or through any other way you find suitable.

I know these contributions might seem small, but I've put a lot of effort into them, balancing this with my company's tasks, where I'm also working on a project using LangGraph. The project has been quite challenging, but I've been learning a lot along the way. Your support in highlighting this achievement would be greatly appreciated.

Thank you for considering my request and for all the incredible work the LangChain team continues to do.

Best regards,
Hassan Memon

@Hassan-Memon Hassan-Memon force-pushed the docs/update-docstring-for-generate-function branch from 6c1ee0b to e14e26b Compare August 21, 2024 01:56
@vbarda
Copy link
Collaborator

vbarda commented Aug 22, 2024

hm, what is your change? i don't see any difference from main

" state (messages): The current state\n",
"\n",
" Returns:\n",
" dict: The updated state, including the LLM's generated answer\n",
Copy link
Contributor Author

@Hassan-Memon Hassan-Memon Aug 22, 2024

Choose a reason for hiding this comment

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

@vbarda, here's the proposed update: The updated state, including the LLM's generated answer

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.

2 participants