search

winkjs / wink-eng-lite-model

English lite language model for wink-nlp.
https://winkjs.org/wink-nlp/
MIT License
11 stars 1 forks source link

npm package wink-eng-lite-model #7

Closed btakita closed 2 years ago

btakita commented 2 years ago

The wink-nlp install script does not work with pnpm monorepos...

❯ node -e "require( 'wink-nlp/models/install' )" wink-eng-lite-model

npm uninstall https://github.com/winkjs/wink-eng-lite-model/releases/download/1.3.1/wink-eng-lite-model-1.3.1.tgz
npm ERR! code EUNSUPPORTEDPROTOCOL
npm ERR! Unsupported URL Type "workspace:": workspace:*

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/brian/.npm/_logs/2022-08-11T14_46_21_368Z-debug-0.log
node:child_process:926
    throw err;
    ^

Error: Command failed: npm uninstall https://github.com/winkjs/wink-eng-lite-model/releases/download/1.3.1/wink-eng-lite-model-1.3.1.tgz
    at checkExecSyncError (node:child_process:851:11)
    at Object.execSync (node:child_process:923:15)
    at Object.<anonymous> (/project/node_modules/.pnpm/wink-nlp@1.12.0/node_modules/wink-nlp/models/install.js:53:14)
    at Module._compile (node:internal/modules/cjs/loader:1120:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1174:10)
    at Module.load (node:internal/modules/cjs/loader:998:32)
    at Module._load (node:internal/modules/cjs/loader:839:12)
    at Module.require (node:internal/modules/cjs/loader:1022:19)
    at require (node:internal/modules/cjs/helpers:102:18) {
  status: 1,
  signal: null,
  output: [ null, null, null ],
  pid: 882155,
  stdout: null,
  stderr: null
}

Node.js v18.7.0

Distributing the wink-eng-lite-model as an npm package should fix this issue.

sanjayaksaxena commented 2 years ago

Hello @btakita

Please use wink-eng-lite-web-model instead. This model works on both web browsers and Nodejs. It requires Nodejs version 16 or above. The wink-eng-lite-model will be on its retirement path soon!

Best, Sanjaya