mem0 icon indicating copy to clipboard operation
mem0 copied to clipboard

Add folder and branch to GitHub

Open Dev-Khant opened this issue 1 year ago • 1 comments

Description

Add support for extraction of data from specific Branch or File.

Fixes #1243

Type of change

Please delete options that are not relevant.

  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [x] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [ ] Refactor (does not change functionality, e.g. code style improvements, linting)
  • [ ] Documentation update

How Has This Been Tested?

Yes

Please delete options that are not relevant.

  • [x] Unit Test
  • [ ] Test Script (please provide)

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
  • [x] I have made corresponding changes to the documentation
  • [x] My changes generate no new warnings
  • [ ] 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
  • [ ] Any dependent changes have been merged and published in downstream modules
  • [x] I have checked my code and corrected any misspellings

Maintainer Checklist

  • [ ] closes #xxxx (Replace xxxx with the GitHub issue number)
  • [ ] Made sure Checks passed

Dev-Khant avatar Mar 08 '24 16:03 Dev-Khant

Error not generated from my changes. Let me know how can I fix it.

Dev-Khant avatar Mar 08 '24 16:03 Dev-Khant

Can you please check why the tests are failing?

deshraj avatar Mar 10 '24 08:03 deshraj

@deshraj Error is caused because of tests/evaluation/test_context_relevancy_metric.py. Locally all the tests are passing.

Screenshot 2024-03-10 at 3 20 19 PM Screenshot 2024-03-10 at 3 19 30 PM

Dev-Khant avatar Mar 10 '24 09:03 Dev-Khant

Oh interesting. Will merge this PR and debug it separately on my end. Thanks for this PR!

deshraj avatar Mar 12 '24 19:03 deshraj