Ticket #26 (closed: fixed)
Create a SaveCSV data handling command
Reported by: | Nick Draper | Owned by: | Anders Markvardsen |
---|---|---|---|
Priority: | major | Milestone: | Iteration 3 |
Component: | Keywords: | ||
Cc: | Blocked By: | ||
Blocking: | Tester: |
Description
Create a data handling command capable of saving to a (comman, tab, user specified) seperated file the contents for a workspace.
Change History
comment:2 Changed 13 years ago by Nick Draper
- Owner set to Anders Markvardsen
Initially you should create a simple implmentation to be able to save a Workspace1D data to file.
After that is working and has a few simple unit tests then you should check it in and start considering a more generic way of saving the data in the workspaces that can also cover the other specialisations of workspaces. Something similar to that used in GAUDI may be a usefull approach eg. having each workspace able to serialise it's data and the SaveCSV command be able to convert the serilised data into a sensinble file format.
comment:5 in reply to: ↑ 1 ; follow-up: ↓ 6 Changed 13 years ago by Anders Markvardsen
Replying to Nick Draper:
Input parameters need to be
InputWorkspace - the name of the workspace containing the data to be saved
Filename - the name of the file to be used
Seperator - the item seperator string default ","
LineSeperator - the seperator between lines default "\n"
comment:6 in reply to: ↑ 5 Changed 13 years ago by Anders Markvardsen
Still getting my head around how to use this ticket system.
The reason for replying to the first comment:1 is to inform that the input property names for the SaveCSV algorithm ended up with slightly different names then originally proposed. The reason for this was to be consistent with the property names already used in the LoadRaw algorithm.
Replying to Anders Markvardsen:
Replying to Nick Draper:
Input parameters need to be
InputWorkspace - the name of the workspace containing the data to be saved
Filename - the name of the file to be used
Seperator - the item seperator string default ","
LineSeperator - the seperator between lines default "\n"
comment:9 Changed 13 years ago by Anders Markvardsen
(In [152]) Added boost library which match the include/boost/filesystem.hpp boost header file version 1_34. Notice the commit text for commit number 146 saids that the boost headers added in that commit are version 1_43. This is a typing error it should read 1_34. For future reference: only the libboost_filesystem.a was added at this point. In addition the .so, multitreaded and/or debugging versions of this library may be added later. Refs #26.
comment:10 Changed 13 years ago by Anders Markvardsen
comment:11 Changed 13 years ago by Anders Markvardsen
comment:13 Changed 13 years ago by Anders Markvardsen
(In [258]) Added test script for the SaveCSV class! It was tested on linuxs1 using Russell runTest script. Also, this test script links to boost_filesystem. A -lboost_filesystem was added to SConstruct in the Builds/Tests directory. Notice currently the Build/Tests/SConstruct is currently failing on the builds, so SaveCSVTest.h was not tested against this SConstruct for this submit. Refs #26.
comment:14 Changed 13 years ago by Anders Markvardsen
comment:15 Changed 13 years ago by Anders Markvardsen
- Status changed from assigned to closed
- Resolution set to fixed
Considering a more generic way of saving the data in the workspaces will be dealt with later.
comment:16 Changed 13 years ago by Anders Markvardsen
- Status changed from closed to reopened
- Resolution fixed deleted
To add boost_filesystem libraries to get SaveCSVTest to pass on build server.
comment:17 Changed 13 years ago by Anders Markvardsen
- Status changed from reopened to closed
- Resolution set to fixed
comment:19 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 874
Input parameters need to be
InputData - the name of the workspace containing the data to be saved
File - the name of the file to be used
Seperator - the item seperator string default ","
LineSeperator - the seperator between lines default "\n"