search

msimet / Stile

Stile: the Systematics Tests In Lensing pipeline
BSD 3-Clause "New" or "Revised" License
9 stars 6 forks source link

File output: simple change #19

Closed rmandelb closed 10 years ago

rmandelb commented 10 years ago

Hironao would like to change the file output so that the filename automatically includes visit/CCD or tract/patch or some other unique-ish identifier, so that plots don't get overwritten when you make a new one for a new CCD. I believe he said this should be a simple change that he can do. Is that right, @HironaoMiyatake ?

HironaoMiyatake commented 10 years ago

Yes, I think so at least for the CCD level. I'll make a change based on branch #7.

HironaoMiyatake commented 10 years ago

I added a simple change for this. For CCD-level tests, a filename is $SUPRIME_DATA_DIR/rerun/[rerun/name/for/stile]/%(pointing)05d/%(filter)s/stile_output/[sys_test_name]-%(visit)07d-%(ccd)03d.png . For field-level tests, a filename is $SUPRIME_DATA_DIR/rerun/[rerun/name/for/stile]/%(pointing)05d/%(filter)s/stile_output/[sys_test_name]-%(visit)07d-[ccds].png , where [ccds] is a reduced form of a CCD list. For example, if a CCD list is [0, 1, 2, 4, 5, 6, 8, 10], [ccds] becomes 0..2^4..6^8^10.

Do we want to merge this change to the PR of #7 , or do we want make a PR for this branch after merging #7 to master?

msimet commented 10 years ago

If you don't mind merging it into #7, let's do that--this is much better!

HironaoMiyatake commented 10 years ago

Done! Shall we close this issue?

msimet commented 10 years ago

Sure! Closing...