Fix confusion in error message #504

Merged
merged 1 commit into from Jan 19, 2016

Projects

None yet

2 participants

@andyfangdz
Contributor

The error message was "missing output_model",
while the variable used for detecting it was "outputFile",
which refers to the testing output, rather than the model output.

@andyfangdz andyfangdz Fix confusion in error message
The error message was "missing output_model",
while the variable used for detecting it was "outputFile",
which refers to the testing output, rather than the model output.
d72958e
@rcurtin
Member
rcurtin commented Jan 19, 2016

Nice catch, thanks! This is a result of some earlier refactoring.

@rcurtin rcurtin merged commit b29997d into mlpack:master Jan 19, 2016

1 check was pending

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
@rcurtin rcurtin added this to the mlpack 2.0.1 milestone Jan 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment