Clojush icon indicating copy to clipboard operation
Clojush copied to clipboard

Remove limit on fields printed in CSV reports

Open NicMcPhee opened this issue 8 years ago • 1 comments

In csv-print there is a filter that significantly limits the set of output fields that can be printed:

(let [columns (concat [:uuid]
                        (filter #(some #{%} csv-columns)
                                [:generation :location :parent-uuids :genetic-operators :push-program-size :plush-genome-size :push-program :plush-genome :total-error]))]

@thelmuth thinks this may have been done to ensure that the columns were printed in the same order. It's quite limiting, though, and it would be nice if we could add other fields on the command line without having to alter this code.

One option is to simply remove the filter, and always use whatever order is provided in the runtime configuration.

Another option would be to keep the filter to maintain the order of the "standard" fields, and then concat on the remaining fields, perhaps sorted by alphabetical order on the keyword.

The first would be simpler and less confusing to people coming to the code, but second wouldn't be hard if people would prefer that approach.

NicMcPhee avatar Sep 29 '15 02:09 NicMcPhee

I'm pretty flexible here. I can't remember why it's there, so there might be a better reason than I'm remembering. Maybe not though!

thelmuth avatar Sep 29 '15 02:09 thelmuth