ratt-ru / simms

Creates empty measurement sets using the the CASA simulate tool.
GNU General Public License v2.0
7 stars 8 forks source link

scan time carve up results in infinite loop and inconsistent total synthesis time #57

Closed bennahugo closed 2 years ago

bennahugo commented 3 years ago

@SpheMakh your logic carving the synthesis time has a few bugs:

I will submit a fix which defines synthesis time as synthesis time per field and place as many complete scans into that synthesis time as possible unless there is only one scan which will be the total synthesis time of that field.

I also add the functionality interleave fields as would be the case for a more realistic observation.

One can always make the scan carve up logic more complicated if you want (ie. split the observation into n number of complete scans plus 1 incomplete scan and place them between fields. If you want I can implement this, but for now I will make a PR to at least provide functionality for the above.