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

Add Langchain Pandas DataFrame Agent #83

Merged
merged 14 commits into from
Oct 23, 2023
Merged

Conversation

MarcSkovMadsen
Copy link
Collaborator

@MarcSkovMadsen MarcSkovMadsen commented Oct 21, 2023

Addresses #79

Todo

  • Implement POC
  • Add .csv file to download in "Welcome" message.
  • Remove rerun, undo and clear buttons
  • Add tests
  • Create video and thumbnail
  • Add docstring
  • Add option to use PanelCallbackHandler to better understand what goes on.

Won't do now. Maybe one day someone will.

Related issues

langchain_chat_pandas_df.mp4

@MarcSkovMadsen MarcSkovMadsen marked this pull request as ready for review October 21, 2023 15:35
@codecov-commenter
Copy link

codecov-commenter commented Oct 21, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (2b801a4) 78.88% compared to head (8903405) 78.88%.
Report is 8 commits behind head on main.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #83   +/-   ##
=======================================
  Coverage   78.88%   78.88%           
=======================================
  Files           5        5           
  Lines         161      161           
=======================================
  Hits          127      127           
  Misses         34       34           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

MarcSkovMadsen and others added 2 commits October 22, 2023 05:09
Co-authored-by: Andrew <15331990+ahuang11@users.noreply.github.com>
@MarcSkovMadsen
Copy link
Collaborator Author

Thanks for the feedback @ahuang11 . I've implemented the changes. Let me know if there is more I should do?

@ahuang11
Copy link
Collaborator

ahuang11 commented Oct 22, 2023

I encounter some issues (duplicates using show chain of thought) using https://github.com/holoviz/panel/pull/5710/files; I'm not sure if it's related to the branch or this.

image

@MarcSkovMadsen
Copy link
Collaborator Author

I encounter some issues (duplicates using show chain of thought) using https://github.com/holoviz/panel/pull/5710/files; I'm not sure if it's related to the branch or this.

image

How can I reproduce this? I see the exception at the top, but does not know the steps you applied to get to that?

@ahuang11 ahuang11 merged commit b433d32 into main Oct 23, 2023
2 of 4 checks passed
@ahuang11
Copy link
Collaborator

Thanks for adding this example!

@ahuang11
Copy link
Collaborator

I encounter some issues (duplicates using show chain of thought) using https://github.com/holoviz/panel/pull/5710/files; I'm not sure if it's related to the branch or this.

image

How can I reproduce this? I see the exception at the top, but does not know the steps you applied to get to that?

I believe I just copied the text from the sample query provided.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants