Drew,

Thanks for this catch.  That was a debugging strategy that got left in
place.

Both of these options are good (temp and target).  I would tend to opt for
tmp files since it is child's play to change it to a preserved location.

And I insist that I should be the one who should feel sheepish here.

On Tue, Oct 12, 2010 at 7:05 PM, Drew Farris <[email protected]> wrote:

> Doing some builds tonight, I noticed TrainLogisticTest is writing its
> output to a directory named model in the examples project directory. I
> suspect we shouldn't write it there, but I'm being a big sheepish
> about 'just fixing' it.
>
> We can either write the model to a temp file using:
>
> getTestTempFile("model").getAbsolutePath();
>
> Or write it to target, e.g: ./target/model -- where it will stick
> around until the next clean.
>
> The latter has the advantage of preserving the model for later
> (manual) inspection while the former keeps things nice and sparking
> clean.
>

Reply via email to