search

mhardcastle / ddf-pipeline

LOFAR pipeline using killms/ddfacet
GNU General Public License v2.0
23 stars 19 forks source link

Stokesv fix #324

Closed twshimwell closed 9 months ago

twshimwell commented 1 year ago

Pull request to a) image each observation in stokes V instead of all together; b) make splitisland size optional; c) make using maskdiffuse optional.

Not fully tested yet...

mhardcastle commented 1 year ago

Have you thought about integration with the uploads/archive for the QUV stuff, Tim?

twshimwell commented 1 year ago

Ah just did the change to rclone_upload.py in a lotss-hba-survey branch (stokes-fix) and put in a pull request there too.

For now ive just done the Stokes V because the products will be small. For the QU ive mailed Vibor and Shane to see if it is 100% needed because those products are going to be v.large...

twshimwell commented 11 months ago

Any thoughts on merging this into master?

mhardcastle commented 10 months ago

This needs a test pipeline run before we merge it.

twshimwell commented 9 months ago

@mhardcastle I finally got around to testing this. Pushed one small typo yesterday but otherwise seems to work as expected. So produces V maps for each epoch, named e.g.:

image_full_high_stokesV_L796616.dirty.fits

mhardcastle commented 9 months ago

OK cool. I will merge, make a new singularity and we'll see if it works in production.

twshimwell commented 9 months ago

Thank you very much!