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.

Sign up for GitHub

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

Jump to bottom

Regression training limit #413

Merged
merged 19 commits into from
Oct 8, 2019
Merged

Regression training limit #413

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
e4b8a92
refactored maxTrainingSample get and set function so that all classes…
AdamChit Sep 20, 2019
2170254
added downsampling logic if MaxTrainingSample reached
AdamChit Sep 20, 2019
dff09b9
added unit tests for downsampling in regression data splitter
AdamChit Sep 20, 2019
14c6b42
added integration tests to test downsampling from the model selector …
AdamChit Sep 20, 2019
722341b
style changes
AdamChit Oct 3, 2019
34d5bf1
changed the test to reduce run time
AdamChit Oct 3, 2019
8e2778d
Merge branch 'master' into achit/regression-training-limit
AdamChit Oct 3, 2019
433d483
test now checks all data splitter params
AdamChit Oct 4, 2019
0932810
Merge branch 'achit/regression-training-limit' of https://github.com/…
AdamChit Oct 4, 2019
2b02f8a
Update core/src/test/scala/com/salesforce/op/stages/impl/tuning/DataS…
AdamChit Oct 4, 2019
0521a37
Update core/src/test/scala/com/salesforce/op/stages/impl/regression/R…
AdamChit Oct 4, 2019
962e06f
added downSampleFraction default value and made style changes
AdamChit Oct 4, 2019
80a80d5
Merge branch 'achit/regression-training-limit' of https://github.com/…
AdamChit Oct 4, 2019
0ab4d9a
renamed test
AdamChit Oct 4, 2019
ef4327c
changed getDownSampleFraction to protected
AdamChit Oct 5, 2019
8ca0e78
name change based on RP comments
AdamChit Oct 5, 2019
8e67f27
added datacount to summary
AdamChit Oct 5, 2019
009706d
Trigger re-build
AdamChit Oct 7, 2019
cfbe22f
Trigger travis re-build
AdamChit Oct 7, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
changed getDownSampleFraction to protected
  • Loading branch information
@AdamChit
AdamChit committed Oct 5, 2019
commit ef4327cbe19035383eec199f802e42543f1e69cc
6 changes: 3 additions & 3 deletions core/src/main/scala/com/salesforce/op/stages/impl/tuning/DataSplitter.scala
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -109,16 +109,16 @@ trait DataSplitterParams extends Params {
*
* @group param
*/
private[op] final val downSampleFraction = new DoubleParam(this, "downSampleFraction",
protected[op] final val downSampleFraction = new DoubleParam(this, "downSampleFraction",
"fraction to down sample data", ParamValidators.inRange(
lowerBound = 0.0, upperBound = 1.0, lowerInclusive = false, upperInclusive = true
)
)
setDefault(downSampleFraction, SplitterParamsDefault.DownSampleFractionDefault)

private[op] def setDownSampleFraction(value: Double): this.type = set(downSampleFraction, value)
protected[op] def setDownSampleFraction(value: Double): this.type = set(downSampleFraction, value)

private[op] def getDownSampleFraction: Double = $(downSampleFraction)
protected[op] def getDownSampleFraction: Double = $(downSampleFraction)
}

/**
Expand Down