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

Falkordb graph documents #6

Closed
wants to merge 21 commits into from
Closed

Conversation

gkorland
Copy link

No description provided.

gkorland and others added 21 commits September 26, 2023 14:52
@baskaryan

---------

Co-authored-by: AashutoshPathakShorthillsAI <[email protected]>
Co-authored-by: Aayush <[email protected]>
Co-authored-by: Aashish Saini <[email protected]>
Co-authored-by: ManpreetShorthillsAI <[email protected]>
Co-authored-by: AryamanJaiswalShorthillsAI <[email protected]>
Co-authored-by: Adarsh Shrivastav <[email protected]>
Co-authored-by: Vishal <[email protected]>
Co-authored-by: ChetnaGuptaShorthillsAI <[email protected]>
Co-authored-by: PankajKumarShorthillsAI <[email protected]>
Co-authored-by: AbhishekYadavShorthillsAI <[email protected]>
Co-authored-by: AmitSinghShorthillsAI <[email protected]>
Co-authored-by: Md Nazish Arman <[email protected]>
Co-authored-by: KamalSharmaShorthillsAI <[email protected]>
Co-authored-by: Lakshya <[email protected]>
Co-authored-by: AnujMauryaShorthillsAI <[email protected]>
Co-authored-by: Saransh Sharma <[email protected]>
Co-authored-by: GhayurHamzaShorthillsAI <[email protected]>
Co-authored-by: Puneet Dhiman <[email protected]>
Co-authored-by: Riya Rana <[email protected]>
Co-authored-by: Akshay Tripathi <[email protected]>
…gchain's dumps() (langchain-ai#10593)

The intermediate steps example in docs has an example on how to retrieve
and display the intermediate steps.
But the intermediate steps object is of type AgentAction which cannot be
passed to json.dumps (it raises an error).
I replaced it with Langchain's dumps function (from langchain.load.dump
import dumps) which is the preferred way to do so.
Fixing a typo in the example code in the docstring...

You have to start somewhere though right?

Co-authored-by: Harrison Chase <[email protected]>
The new Fireworks and FireworksChat implementations are awesome! Added
in this PR langchain-ai#11117 thank
you @ZixinYang

However, I think stop words were not plumbed correctly. I've made some
simple changes to do that, and also updated the notebook to be a bit
clearer with what's needed to use both new models.


---------

Co-authored-by: Taqi Jaffri <[email protected]>
…n-ai#9318)

- Description: this PR adds the support for arxiv identifier of the
ArxivAPIWrapper. I modified the `run()` and `load()` functions in
`arxiv.py`, using regex to recognize if the query is in the form of
arxiv identifier (see
[https://info.arxiv.org/help/find/index.html](https://info.arxiv.org/help/find/index.html)).
If so, it will directly search the paper corresponding to the arxiv
identifier. I also modified and added tests in `test_arxiv.py`.
  - Issue: langchain-ai#9047 
  - Dependencies: N/A
  - Tag maintainer: N/A

---------

Co-authored-by: Bagatur <[email protected]>
Co-authored-by: Harrison Chase <[email protected]>
This removes the use of the intermediate df list and directly
concatenates the dataframes if path is a list of strings. The pd.concat
function combines the dataframes efficiently, making it faster and more
memory-efficient compared to appending dataframes to a list.

<!-- Thank you for contributing to LangChain!

Replace this comment with:
  - Description: a description of the change, 
  - Issue: the issue # it fixes (if applicable),
  - Dependencies: any dependencies required for this change,
- Tag maintainer: for a quicker response, tag the relevant maintainer
(see below),
- Twitter handle: we announce bigger features on Twitter. If your PR
gets announced and you'd like a mention, we'll gladly shout you out!

Please make sure you're PR is passing linting and testing before
submitting. Run `make format`, `make lint` and `make test` to check this
locally.

If you're adding a new integration, please include:
1. a test for the integration, preferably unit tests that do not rely on
network access,
  2. an example notebook showing its use.

Maintainer responsibilities:
  - General / Misc / if you don't know who to tag: @baskaryan
  - DataLoaders / VectorStores / Retrievers: @rlancemartin, @eyurtsev
  - Models / Prompts: @hwchase17, @baskaryan
  - Memory: @hwchase17
  - Agents / Tools / Toolkits: @hinthornw
  - Tracing / Callbacks: @agola11
  - Async: @agola11

If no one reviews your PR within a few days, feel free to @-mention the
same people again.

See contribution guidelines for more information on how to write/run
tests, lint, etc:
https://github.com/hwchase17/langchain/blob/master/.github/CONTRIBUTING.md
 -->

---------

Co-authored-by: Harrison Chase <[email protected]>
Reverts langchain-ai#8610

this is actually an oversight - this merges all dfs into one df. we DO
NOT want to do this - the idea is we work and manipulate multiple dfs
### Description: 
NotionDB supports a number of common property types. I have found three
common types that are not included in notiondb loader. When programs
loaded them with notiondb, which will cause some metadata information
not to be passed to langchain. Therefore, I added three common types:
- date
- created_time
- last_edit_time.

### Issue: 
no
### Dependencies: 
No dependencies added :)
### Tag maintainer: 
@rlancemartin, @eyurtsev
### Twitter handle: 
@BJTUTC
- **Description:** Prompt wrapping requirements have been implemented on
the service side of AWS Bedrock for the Anthropic Claude models to
provide parity between Anthropic's offering and Bedrock's offering. This
overnight change broke most existing implementations of Claude, Bedrock
and Langchain. This PR just steals the the Anthropic LLM implementation
to enforce alias/role wrapping and implements it in the existing
mechanism for building the request body. This has also been tested to
fix the chat_model implementation as well. Happy to answer any further
questions or make changes where necessary to get things patched and up
to PyPi ASAP, TY.
- **Issue:** No issue opened at the moment, though will update when
these roll in.
  - **Dependencies:** None

---------

Co-authored-by: Harrison Chase <[email protected]>
Co-authored-by: Bagatur <[email protected]>
@gkorland gkorland closed this Sep 29, 2023
@gkorland gkorland deleted the falkordb-graph_documents branch September 29, 2023 12:06
gkorland pushed a commit that referenced this pull request Oct 17, 2023
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.