-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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 options is undefined! #280
Conversation
src/cross-validate.js
Outdated
@@ -116,7 +116,7 @@ export default class CrossValidate { | |||
let dclone = data.slice(0); | |||
let testSet = dclone.splice(i * size, size); | |||
let trainSet = dclone; | |||
let result = this.testPartition(options, trainOpts, trainSet, testSet); | |||
let result = this.testPartition(this.options, trainOpts, trainSet, testSet); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice find! We could (should) simplify this. We don't need to pass options to testPartition
because it is part of this
, can you remove here and reference there?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I take it back, this.options
just needs removed in this case. I'll go ahead and commandeer this as I need it for work this morning.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
Ty sir! |
#185 (comment)