-
Notifications
You must be signed in to change notification settings - Fork 40
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Error during npm install #76
Comments
Let me take a look. |
Looks like I got past this by directly doing an npm install on gcc and
make.
…On Wed, Jun 12, 2024 at 12:22 PM Adhityan K V ***@***.***> wrote:
Let me take a look.
—
Reply to this email directly, view it on GitHub
<#76 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AH7CHDBHSJDZUEPLYZ47TZ3ZHCNYRAVCNFSM6AAAAABJG6AF4KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRTG42DINJTGU>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
This looks like an issue with one of the dependencies. They have a PR addressing this but it has not been merged for months (mscdex/mmmagic#171). I will try and get the upstream patched; if not, I will publish a version using alternative package or a forked version. |
The install of mmmagic works for me but then when I run the app I created I get:
I notice there is no Release directory inside node_modules/mmmagic/build directory. Is that expected? |
Published a new version ( |
So the install worked, but still having issues using things. I am trying to install this in a nextjs app by the way. So now the npm install works. Then I did |
What's interesting is that if I change the import to
|
Yes, the import path is Actually, NextJs has some issues with node native packages. I am currently actually testing this out and building a sample app here. It's still a WIP but I hope to get support added soon. |
Thanks for the update. I think I’ll check back when this works for nextjs.
…On Fri, Jun 14, 2024 at 7:09 AM Adhityan K V ***@***.***> wrote:
Yes, the import path is import { HNSWDb } from
***@***.***/embedjs/vectorDb/hnswlib'. You can refer to all the import
paths in the documentation
<https://github.com/llm-tools/embedJs?tab=readme-ov-file#hnswlib>.
Actually, NextJs has some issues with node native packages. I am currently
actually testing this out and building a sample app here
<https://github.com/llm-tools/chat-bot-nextjs-template>. It's still a WIP
but I hope to get support added soon.
—
Reply to this email directly, view it on GitHub
<#76 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AH7CHDCMFTRKGVELQLTSJ3DZHLFP5AVCNFSM6AAAAABJG6AF4KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRXG44TMMRQGE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Will post an update when NextJs is supported. Closing the issue for now. |
So finally have this working with NextJs. Check the example here with pinecone - https://github.com/llm-tools/chat-bot-nextjs-template. Point to note - not all vector databases work with NextJs. It may be possible to get each of them wotking with specific config changes to the bundler but by default some of them will not work. The base library itself will work fine out of the box. Beyond that, several API based vector databases work out of the box. |
Trying to install the package on my mac and get the following error:
The text was updated successfully, but these errors were encountered: