Skip to content

Create alternative_main_with_pii.py#1

Open
dganesh05 wants to merge 1 commit into
masterfrom
patch-1
Open

Create alternative_main_with_pii.py#1
dganesh05 wants to merge 1 commit into
masterfrom
patch-1

Conversation

@dganesh05

@dganesh05 dganesh05 commented Oct 13, 2024

Copy link
Copy Markdown
Collaborator

This is a longer alternative to main.py, grouping essential processes into customizable functions whose parameters are explained via docstring. This allows for greater degree of experimentation such as adjusting text splitter parameters, retriever parameters, and most imporantly, models used for embedding and generation. The code also includes a function for censoring pii, which is done after loading the document and before creating the embeddings.

My source of reference is as follows:
Langchain tutorial: https://youtu.be/yF9kGESAi3M?feature=shared Github repo used: https://github.com/bhancockio/langchain-crash-course (Only need to review the sections of 4-rag)

I would like to note that the llm would vary depending on what the classification team comes up with. It is temporary, and could be integrated into main.py with changes depending on the llm and the unit tests involved.

This is a longer alternative to main.py, grouping essential processes into customizable functions whose parameters are explained via docstring. This allows for greater degree of experimentation such as adjusting text splitter parameters, retriever parameters, and most imporantly, models used for embedding and generation. The code also includes a function for censoring pii, which is done after loading the document and before creating the embeddings.

My source of reference is as follows:
Langchain tutorial: https://youtu.be/yF9kGESAi3M?feature=shared
Github repo used: https://github.com/bhancockio/langchain-crash-course
(Only need to review the sections of 4-rag)
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.

1 participant