Free cookie consent management tool by TermsFeed Policy Generator

Opened 12 years ago

Closed 12 years ago

#1922 closed enhancement (done)

Improve CSV export

Reported by: abeham Owned by: gkronber
Priority: medium Milestone: HeuristicLab 3.3.8
Component: Problems.Instances Version: 3.3.8
Keywords: Cc:

Description (last modified by abeham)

The CSV export should use the system defined list separator and not hard-code to use semi-colon. Additionally, it should output all variables in the dataset, not just those containing double values.

Change History (10)

comment:1 Changed 12 years ago by abeham

  • Status changed from new to accepted

comment:2 Changed 12 years ago by abeham

  • Owner changed from abeham to sforsten
  • Status changed from accepted to reviewing

r8530: Used the culture's defined list separator in CSV export

comment:3 Changed 12 years ago by sforsten

  • Owner changed from sforsten to abeham
  • Status changed from reviewing to readytorelease

Tested and works

comment:4 Changed 12 years ago by gkronber

  • Status changed from readytorelease to reviewing

Also reviewed r8530. The change should be OK, however, there could be a problem when the variable name contains the list separator character. This should be rather easy to detect and handle correctly. Could you please add this?

comment:5 Changed 12 years ago by abeham

  • Status changed from reviewing to assigned

comment:6 Changed 12 years ago by abeham

  • Status changed from assigned to accepted

comment:7 Changed 12 years ago by abeham

  • Description modified (diff)
  • Summary changed from Use current culture's list separator as separator in csv export to Improve CSV export
  • Type changed from defect to enhancement

comment:8 Changed 12 years ago by abeham

  • Owner changed from abeham to gkronber
  • Status changed from accepted to reviewing

r8685:

  • Removed list separator in variable names if present
  • Changed export to output all variables in the dataset

comment:9 Changed 12 years ago by gkronber

  • Status changed from reviewing to readytorelease

r8685 looks good, thanks!

comment:10 Changed 12 years ago by swagner

  • Resolution set to done
  • Status changed from readytorelease to closed
  • Version changed from 3.3.7 to 3.3.8
Note: See TracTickets for help on using tickets.