zachmayer / caretEnsemble

caret models all the way down :turtle:
Other
226 stars 75 forks source link

Fix ordering by y(obs) issue #235

Closed the-tourist- closed 6 years ago

the-tourist- commented 6 years ago

Instead of keeping the original ordering caretEnsemble has been ordering the x and y by the value of y (the obs). Fixed to order by rowIndex instead.

lintr-bot commented 6 years ago

R/helper_functions.R:332:1: style: Trailing whitespace is superfluous.

​  
^~
coveralls commented 6 years ago

Coverage Status

Changes Unknown when pulling c2b90018d3ab3125f47c0a63ab1dc64b58004c9e on the-tourist-:master into on zachmayer:master.

zachmayer commented 6 years ago

Please fix the lint issue, and then I can merge:

R/helper_functions.R:332:1: style: Trailing whitespace is superfluous.

​  
^~
coveralls commented 6 years ago

Coverage Status

Changes Unknown when pulling 3367597d40813e295e4bc295edd24d56c3f58457 on the-tourist-:master into on zachmayer:master.