kevinlu1248 / llama-hub

A library of data loaders for LLMs made by the community -- to be used with GPT Index and/or LangChain
MIT License
0 stars 0 forks source link

[Bug Fix] Fix issues with PandasExcelReader #3

Open sweep-ai[bot] opened 1 year ago

sweep-ai[bot] commented 1 year ago

Description

This PR fixes two issues with the PandasExcelReader class in the llama_hub/file/pandas_excel/base.py file.

  1. The load_data method did not accept a pandas_config argument, causing a TypeError when trying to pass this argument. This PR modifies the load_data method to accept the pandas_config argument.

  2. The __init__ method of the PandasExcelReader class did not correctly initialize the _row_joiner attribute, causing an AttributeError when trying to access this attribute. This PR ensures that the _row_joiner attribute is correctly initialized in the __init__ method.

Changes Made

Testing

I have tested the modified PandasExcelReader class by running the provided example code and confirming that it no longer throws any errors. I have also ensured that the existing functionality of the class is not affected by these changes.

Related Issue

This PR addresses the issue #1.

Checklist

Fixes #1.

To checkout this PR branch, run the following command in your terminal:

git checkout sweep/fix-pandas-excel-reader