mhardcastle / ddf-pipeline

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

revisiting ssd maxisland #310

Closed twshimwell closed 1 year ago

twshimwell commented 1 year ago

Presently we have --SSDClean-MaxIslandSize 100 for galactic fields which splits large islands into regions of 100x100 pixels but it seems we can change this to something much larger.

For a galactic test field with large regions of diffuse emission ive been using --SSDClean-MaxIslandSize 0 which doesnt do any splitting and this seems to work pretty well even for very large islands. I think we introduced --SSDClean-MaxIslandSize 100 because we kept running into memory issues when it had it at 0. Wondering though if we should revisit this and perhaps keep it at 0 for all fields (presently we use 0 for extragalactic and 100 for galactic). Attached shows comparison models of a bright fairly extended galactic source.

Screenshot 2023-04-17 at 10 42 38

twshimwell commented 1 year ago

repeated issue from #321