PaxInstruments / t100-software

0 stars 1 forks source link

Save T100 temperatures to CSV file #1

Closed charlespax closed 9 years ago

charlespax commented 10 years ago

The CSV should have a timestamp and each TC value.

Time, TC1, TC2,...,TCn

kehribar commented 9 years ago

We have the basic support with https://github.com/PaxInstruments/t100-software/commit/94e99cfc54a52235f80bc79dac77a45685a013d4, but will work more on the future.

For multiple, or actually arbitrary number of, devices; what should be the file content format?

Time_1,Id1,Data,Data,Data ... Time_1,Id2,Data,Data,Data ... Time_1,Id3,Data,Data,Data ... Time_2,Id1,Data,Data,Data ... Time_2,Id2,Data,Data,Data ... Time_2,Id3,Data,Data,Data ...

How does that look?

charlespax commented 9 years ago

The format should list all the temperatures on a single line for each time.

Normal display:

Time, T <serial number T1>,  T <serial number T2>,..., T <serial number Tn>
t1, TT1, TT2,...,TTn
t2, TT1, TT2,...,TTn
...
tn, TT1, TT2,...,TTn

where tn is time at n and TTn is the tip temperature of thermocouple Tn.

If "Record junction temperatures" is checked"

Time, T <serial number T1>, JT <serial number T1>,..., T <serial number Tn>, JT <serial number Tn>
t1, TT1, JT1, TT2, JT2,...,TTn,JTn
t2, TT1, JT1, TT2, JT2,...,TTn,JTn
...
tn, TT1, JT1, TT2, JT2,...,TTn,JTn

where tn is time at n, JTn is junction temperature of thermocouple Tn, and TTn is the tip temperature of thermocouple Tn.

This way a user can open the data in a spreadsheet program, select two columns, and make plot. Putting each

kehribar commented 9 years ago

Should I add "Record ADC values" checkbox as well? You've mentioned this in https://github.com/PaxInstruments/t100-software/issues/11#issuecomment-104224229. ;

Also,

Time, T <serial number T1>, T <serial number T2>,..., T <serial number Tn>

is a separate line in the CSV file, like a header, right?

charlespax commented 9 years ago

Yes, there should be a header at the beginning of the CSV. The line directly above the data should be the names of each column. Above that line I think we can also add stuff. Maybe software and hardware information.

Go ahead and implement having the ADC values in the CSV if it's easy. Otherwise, we should make another issue and do it in the future.

kehribar commented 9 years ago

Initial support with https://github.com/PaxInstruments/t100-software/commit/ef19487cc2045cad171a201c26ae2fd7f923d2a8

Optional readings are added with https://github.com/PaxInstruments/t100-software/commit/9f211eea45a3af754a98db5bbeeb3bbe55d7ebe1

charlespax commented 9 years ago

screen shot 2015-05-26 at 19 05 33