mem0 icon indicating copy to clipboard operation
mem0 copied to clipboard

[Feature] Add Excel Loader to load .xlsx

Open eren-darici opened this issue 1 year ago • 1 comments

Description

  • Excel Loader created
  • Dependencies added
  • Basic tests are created

How Has This Been Tested?

  • Created test_excel.py which includes multi-sheet validation.

Checklist:

  • [x] My code follows the style guidelines of this project
  • [x] I have performed a self-review of my own code
  • [x] I have commented my code, particularly in hard-to-understand areas
  • [ ] I have made corresponding changes to the documentation
  • [x] My changes generate no new warnings
  • [x] I have added tests that prove my fix is effective or that my feature works
  • [x] New and existing unit tests pass locally with my changes
  • [x] Any dependent changes have been merged and published in downstream modules
  • [x] I have checked my code and corrected any misspellings

Maintainer Checklist

  • [x] closes most issues related Excel
  • [ ] Made sure Checks passed

eren-darici avatar Jan 08 '24 20:01 eren-darici

LGTM!

@eren-darici Could you please resolve the conflicts? Also, it would be great it you can add docs for this here. I will merge this and release it in next package release.

deven298 avatar Jan 20 '24 06:01 deven298

Seems like the PR has been inactive. Closing this now. Feel free to reopen if conflicts are resolved.

deshraj avatar Jun 14 '24 06:06 deshraj