mysamai / natural-brain

A natural language classifier using Node Natural with a BrainJS neural network
MIT License
320 stars 29 forks source link

An in-range update of brain.js is breaking the build 🚨 #43

Closed greenkeeper[bot] closed 5 years ago

greenkeeper[bot] commented 5 years ago

The dependency brain.js was updated from 1.4.0 to 1.4.1.

🚨 View failing branch.

This version is covered by your current version range and after updating it in your project the build failed.

brain.js is a direct dependency of this project, and it is very likely causing it to break. If other packages depend on yours, this update is probably also breaking those in turn.

Status Details - ❌ **continuous-integration/travis-ci/push:** The Travis CI build could not complete due to an error ([Details](https://travis-ci.org/mysamai/natural-brain/builds/434227048?utm_source=github_status&utm_medium=notification)).

Commits

The new version differs by 5 commits.

  • f0a1a56 Merge pull request #280 from BrainJS/mubaidr-patch-1
  • 133f40a fix: build
  • dd1ae57 fix: Remove reference to options or this.options to testPartition
  • 812f014 Merge branch 'v1.x' into mubaidr-patch-1
  • 79096ba Fix options is undefined!

See the full diff

FAQ and help There is a collection of [frequently asked questions](https://greenkeeper.io/faq.html). If those don’t help, you can always [ask the humans behind Greenkeeper](https://github.com/greenkeeperio/greenkeeper/issues/new).

Your Greenkeeper Bot :palm_tree:

greenkeeper[bot] commented 5 years ago

After pinning to 1.4.0 your tests are still failing. The reported issue might not affect your project. These imprecisions are caused by inconsistent test results.

greenkeeper[bot] commented 5 years ago

Your tests are passing again with this update. Explicitly upgrade to this version πŸš€

Commits

The new version differs by 2 commits.

  • fc48ae6 Merge pull request #281 from BrainJS/185-cross-validation-fixes
  • ca437f3 fix: Fix CrossValidate to have tests for when data too small

See the full diff