jeffheaton / encog-java-core

http://www.heatonresearch.com/encog
Other
742 stars 268 forks source link

EncogUtility.convertCSV2Binary bug #259

Open shikhirsingh opened 4 years ago

shikhirsingh commented 4 years ago

EncogUtility.convertCSV2Binary does not use the header parameter. False is hard coded in the logic.

    public static void convertCSV2Binary(final File csvFile,
            final File binFile, final int inputCount, final int outputCount,
            final boolean headers) {
        binFile.delete();
        final CSVNeuralDataSet csv = new CSVNeuralDataSet(csvFile.toString(),
                inputCount, outputCount, false); // BUG HERE
        final BufferedMLDataSet buffer = new BufferedMLDataSet(binFile);
        buffer.beginLoad(inputCount, outputCount);
        for (final MLDataPair pair : csv) {
            buffer.add(pair);
        }
        buffer.endLoad();
    }