LSSTDESC / Twinkles

10 years. 6 filters. 1 tiny patch of sky. Thousands of time-variable cosmological distance probes.
MIT License
13 stars 12 forks source link

Added script in bin/get_twinkles_releases.py to produce yearly release #456

Closed rbiswas4 closed 7 years ago

rbiswas4 commented 7 years ago

visits and visists in coadds.

Fixes #454

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 41.35% when pulling f81b0dc05b4b26505bb9e0c2a339598bdd632bb3 on Issue#454/twinkles_data_releases into 1502aa60b02ed4a75ec7d9f9c65d496b835fd963 on master.

drphilmarshall commented 7 years ago

Thanks Rahul! Looking at the numbers of visits in each release: image

I think:

Question:

NB. It makes the file system accounting slightly easier if you number data releases as 01, 02, 03 etc so that they ls nicely...

What do you think?

drphilmarshall commented 7 years ago

PS. You can remake that plot by pasting numbers into this spreadsheet. I did this:

wc -l year_?_Release.csv year_10_Release.csv | grep -v total | awk '{print $1}'

and then just pasted in the resulting column of values.

rbiswas4 commented 7 years ago

@drphilmarshall : Now there are three files for each 'release year'. a DIAvisit file, a coadd file, and a template file. The numbering scheme includes padding for better accounting. This also means that your awk command needs '??' in place of '?' aside from the name change.

Question: Are you including visits from both the DDF proposal and the WFD proposal? It'd be nice to make a WFD set too, so that we can make an error model for the WFD photometric SN sample as well as the DDF spectro/photometric one. I guess it's fair to include both WFD and DDF visits in a "DDF" dataset (because its not like we will ignore the WFD visits when analyzing the real DDFs!) - but the WFD one shoudl contain only WFD proposal visits. Would it be easy to make eg Year_03_WFD_DIAvisits.csv as well as Year_03_DDF_DIAvisits.csv etc?

Good point. And it made me realize I had a bug (which I fixed).

I created a WFD set separately. It is possible (without a whole lot of work) to create a DDF set, but I feel we should consider WFD + DDF togther. So the two sets I made are WFD and Combined.

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 41.35% when pulling 6584c845478d06be74e53d75152930fc69374fc0 on Issue#454/twinkles_data_releases into 1502aa60b02ed4a75ec7d9f9c65d496b835fd963 on master.

drphilmarshall commented 7 years ago

Excellent, thanks Rahul! I think this looks good - see plots below, and I'll merge this in. Thank you! :-)

@tony-johnson Some additional notes for you:

@tony-johnson Do you need anything else from Rahul or Simon to get started (apart from more NERSC hours and Tony-time)? cc @SimonKrughoff

image

image