Andor server change saving format

fanmingyu212 commented 6 years ago

Addresses #220.

This PR introduces following changes:

Config option save_in_sub_dir: If enabled. it will save images in dated folders, such as images_folder/2018/03/2018_03_17. Default enabled.
Config option save_format: It has three choices: "tsv" will save images as tab separated ascii data files, "csv" will save images as comma separated ascii data files, and "bin" will save images as binary file, with each word (2 bytes) representing a pixel in int16 format. The image file extension will be changed to "csv", "tsv", and "dat" corresponding to the save format chosen. Default is "tsv".
Config option save_header: If enabled, it will save headers containing exposure time and em gain information to the ascii data file ("csv" or "tsv"). It reads exposure time and em gain information from gui. If saved in binary format, save_header is ignored. Default is enabled. A example header looks like:
```
# em_gain = 2
# exposure_time = 0.1
123, 124, 125, 162, ... (data)
```
Data files are now saved in integer formats.

jayich commented 6 years ago

Looks fine to me, but @aransfor should review.

aransfor commented 6 years ago

@fanmingyu212 I rarely use the camera server to take data and when I do I grab numpy arrays from the server and save them to datavault as numpy binary files (these are small). I need to have @gkchen and @theoriginaljuice check this pull request since they use the camera for actual experiments. I notified them today

fanmingyu212 commented 6 years ago

@gkchen Could you test the code and give a review of the code when you have time? The code works for @theoriginaljuice and me, so I think we can merge it if it works for you.

gkchen commented 6 years ago

Works for us

On Wed, Mar 21, 2018 at 5:21 PM fanmingyu212 notifications@github.com wrote:

@gkchen https://github.com/gkchen Could you test the code and give a review of the code when you have time? The code works for @theoriginaljuice https://github.com/theoriginaljuice and me, so I think we can merge it if it works for you.

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/CampbellGroup/common/pull/223#issuecomment-375138285, or mute the thread https://github.com/notifications/unsubscribe-auth/AHuAOp9-FzwO_vDh-caiLSMwDNW63iVjks5tgu59gaJpZM4Su9fd .

--

Regards,

Gary Chen

fanmingyu212 commented 6 years ago

@theoriginaljuice @gkchen Thanks for the reviews. I will merge it now.

CampbellGroup / common

Andor server change saving format #223