-
Notifications
You must be signed in to change notification settings - Fork 736
Conversation
@logan-markewich TIA |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice! one comment. also can you remove poetry.lock from being checked in?
llama_hub/bagel/README.md
Outdated
Here's an example usage of the BagelReader. | ||
|
||
```python | ||
from llama_index import download_loader |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: canonical is now from llama_hub.bagel.base import BagelReader
(no need for download_loader)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed in ReadMe..
Thanks
Done.. |
got it.. fixed it.. sorry for the inconvenience. passes the tests. can you merge? |
@logan-markewich can you please look into it?? |
Added loader for BageDB
BagelDB vector store and reader already available at Llama Index.