evalml icon indicating copy to clipboard operation
evalml copied to clipboard

Update split_data with new default for timeseries

Open eccabay opened this issue 2 years ago • 1 comments

Reduces the default test_size in split_data to be 0.1 instead of 0.2, as long as the new value is greater than the passed in forecast horizon.

eccabay avatar Aug 09 '22 18:08 eccabay

Codecov Report

Merging #3650 (874f81a) into main (994d33a) will increase coverage by 0.1%. The diff coverage is 100.0%.

@@           Coverage Diff           @@
##            main   #3650     +/-   ##
=======================================
+ Coverage   99.7%   99.7%   +0.1%     
=======================================
  Files        335     335             
  Lines      33790   33840     +50     
=======================================
+ Hits       33662   33712     +50     
  Misses       128     128             
Impacted Files Coverage Δ
evalml/preprocessing/utils.py 100.0% <100.0%> (ø)
evalml/tests/conftest.py 97.9% <100.0%> (+0.1%) :arrow_up:
...valml/tests/preprocessing_tests/test_split_data.py 100.0% <100.0%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codecov[bot] avatar Aug 09 '22 19:08 codecov[bot]