Skip to content
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

Fix for #209, #222, #372, and #386 # ENOENT, ENOTEMPTY, TypeError: Cannot read properties of undefined (reading 'split') #373

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

watarungurunnn
Copy link

@watarungurunnn watarungurunnn commented Apr 9, 2023

This fixes #209, #222, #372, and #386

@watarungurunnn watarungurunnn changed the title Fix for #222 # ENOTEMPTY: directory not empty Fix for #209, #222 and #372 # ENOENT: no such file or directory and ENOTEMPTY: directory not empty Apr 9, 2023
@watarungurunnn watarungurunnn changed the title Fix for #209, #222 and #372 # ENOENT: no such file or directory and ENOTEMPTY: directory not empty Fix for #209, #222 and #372 # ENOENT, ENOTEMPTY, TypeError: Cannot read properties of undefined (reading 'split') Apr 9, 2023
@erezsh
Copy link

erezsh commented Apr 9, 2023

That solved it for me.

add error handling when being requested w/o model
set the initial value of the model to the first model in the installed models.
@watarungurunnn watarungurunnn changed the title Fix for #209, #222 and #372 # ENOENT, ENOTEMPTY, TypeError: Cannot read properties of undefined (reading 'split') Fix for #209, #222, #372, and #386 # ENOENT, ENOTEMPTY, TypeError: Cannot read properties of undefined (reading 'split') Apr 11, 2023
@InnoVetrov
Copy link

Fixed the issue here as well, got the "reading 'split'" problem on Ubuntu 22.04.2 LTS, and this seem to have fixed it.

@Topzie
Copy link

Topzie commented Apr 13, 2023

I got this problem (iMac). What am I supposed to do to fix this?

/Users/ik/.npm/_npx/3c737cbb02d79cc9/node_modules/dalai/index.js:219
let [Core, Model] = req.model.split(".")
^

TypeError: Cannot read properties of undefined (reading 'split')
at Dalai.query (/Users/ik/.npm/_npx/3c737cbb02d79cc9/node_modules/dalai/index.js:219:35)
at Socket. (/Users/ik/.npm/_npx/3c737cbb02d79cc9/node_modules/dalai/index.js:534:20)
at Socket.emit (node:events:513:28)
at Socket.emitUntyped (/Users/ik/.npm/_npx/3c737cbb02d79cc9/node_modules/socket.io/dist/typed-events.js:69:22)
at /Users/ik/.npm/_npx/3c737cbb02d79cc9/node_modules/socket.io/dist/socket.js:703:39
at process.processTicksAndRejections (node:internal/process/task_queues:77:11)

Node.js v18.16.0

@watarungurunnn
Copy link
Author

watarungurunnn commented Apr 14, 2023

@Topzie As for now, install my repo by replacing dalai to https://github.com/watarungurunnn/dalai, like npx https://github.com/watarungurunnn/dalai llama install 13B or installing it by npx install https://github.com/watarungurunnn/dalai.

@Topzie
Copy link

Topzie commented Apr 15, 2023

Thank you very much, can confirm it's working now.

@BossDaily
Copy link

BossDaily commented Apr 21, 2023

@watarungurunnn I ran npx https://github.com/watarungurunnn/dalai llama install 7B and I still get the same issue.
Do I uninstall the other dalai ?

@watarungurunnn
Copy link
Author

@BossDaily
Yes, try uninstalling dalai, clear the cache and then run my repo.
npx clear-npx-cache

@BossDaily
Copy link

@watarungurunnn Doing npx github:watarungurunnn/dalai serve does nothing

@BossDaily
Copy link

BossDaily commented Apr 21, 2023

wait, never mind I spelled the command wrong in my terminal
Now my current issue running the command produces this result ↓
image
and nothing happens after it is installed
doing the command again I get the same result

@BossDaily
Copy link

okay I got it working by doing npm exec https://github.com/watarungurunnn/dalai serve

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Keep getting an error about "no such file or directory", I dont understand?
6 participants