Jon Saginaw

Results 11 comments of Jon Saginaw

Note: the checks are failing since Motor would be a new package. @skcoirz would love to have you review this PR given our previous convos on async loaders. Thanks!

@skcoirz thanks for all the feedback! I can go ahead and resolve the convos above as it seems we're in alignment here. One question I did have: the build fails...

> @skcoirz I actually copied this approach locally but still get the mypy issue. I didn't want to push another change bc you had mentioned we should be good to...

To resolve the mypy issue, I read their docs about either installing stubs or ignoring the import line. I saw that many other modules in Langchain use type: ignore so...

> @saginawj Hi , could you, please, resolve the merging issues? After that ping me and I push this PR for the review. Thanks! Hi @leo-gan! I resolved the merge...

> @saginawj Please, check the merging files. It seems, there are too many files in this PR. Something went wrong? @leo-gan yes since this PR was from over 4 months...

> lets an example notebook! otherwise looks pretty good Here's a [sample notebook](https://colab.research.google.com/drive/15qQA24fEREoTgzdWKALOcAxXA4ePNsys?usp=sharing) on my repo. In this example, I ask about popular attributes for NFTs in the specified contract...

@hwchase17 made the following updates to this PR: - added multi-blockchain support (Ethereum Testnet, Polygon Mainnet, Polygon Testnet) - added pagination support for the Alchemy API - added notebook to...

Hi @jeffchuber and team. Not sure if this issue is still being worked or if it's considered resolved, but I'm running into the same issues as the rest of the...

@thefedoration thanks for looking at this! and @jeffchuber thanks for prioritizing. I'd love to see a working version pulled in.