search

mljs / ml

Machine learning tools in JavaScript
MIT License
2.63k stars 210 forks source link

Error importing ml-matrix #163

Closed cristianlai closed 4 years ago

cristianlai commented 4 years ago

While running node knn.js, I get the following error: ml-matrix/matrix.js:930 ); ^ SyntaxError: Unexpected token )

It seems because of the list of arguments end with a comma before closing )

Is it a versioning issue?

Thanks in advance.

cristianlai commented 4 years ago

It seems an issue related to trailing comma. Node core would support trailing comma by default when V8 turns it on by default.