search

lmorabit / lofar-vlbi

GNU General Public License v3.0
16 stars 13 forks source link

RA_LOTSS or LOTSS_RA? #63

Closed mooneyse closed 4 years ago

mooneyse commented 4 years ago

skynet.py expects 'RA_LOTSS' in the delay catalogue. However, parallel_split.py expects 'LOTSS_RA' (and likewise for declination).

https://github.com/lmorabit/long_baseline_pipeline/blob/afd3df8ecb26b72bf96c35f65a58b55ce47aa7d3/bin/parallel_split.py#L101

I won't change either as I'm not sure which is the desired label, so I'm just flagging it for others.

lmorabit commented 4 years ago

parallel_split.py is no longer used on the master branch -- do you need this updated still?

mooneyse commented 4 years ago

Nope, thanks :+1: