search

meerklass / museek

A flexible and easy-to-extend data processing pipeline for multi-instrument autocorrelation radio experiments
GNU General Public License v3.0
3 stars 3 forks source link

sanity check should plot scan tracks of more than one antennas #94

Open piyanatk opened 1 week ago

piyanatk commented 1 week ago

At the moment, the sanity check plug-in takes a reference_receiver_index parameter, which is basically the antenna/dish number, e.g. reference_receiver_index=0 refers to m001. Only the scan tracks of this antenna is plotted, but sometimes this antenna turns out to be a strangler, or something happened, showing an incomplete scan track. Below is an example from 1720370484 plot_sanity_check_observation_0 plot_sanity_check_observation_2

Thus, the sanity check should ideally plot scan tracks of more than one antennas for this reason.

mariogrs commented 1 week ago

I agree. Perhaps we could plot the median instead? If the majority of dishes are behaving as expected the median would give the correct path.

On Thu, 5 Sept 2024 at 09:37, Piyanat (Boom) Kittiwisit < @.***> wrote:

At the moment, the sanity check plug-in takes a reference_receiver_index parameter, which is basically the antenna/dish number, e.g. reference_receiver_index=0 refers to m001. Only the scan tracks of this antenna is plotted, but sometimes this antenna turns out to be a strangler, or something happened, showing an incomplete scan track. Below is an example from 1720370484 plot_sanity_check_observation_0.png (view on web) https://github.com/user-attachments/assets/5b275bce-4c76-4137-85c9-2f37461f582d plot_sanity_check_observation_2.png (view on web) https://github.com/user-attachments/assets/544aa424-4ff1-4536-ad74-5b05018a3dee

Thus, the sanity check should ideally plot scan tracks of more than one antennas for this reason.

— Reply to this email directly, view it on GitHub https://github.com/meerklass/museek/issues/94, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB4WJE4L43CCLVQHJKGL4ADZVAJ5JAVCNFSM6AAAAABNV65CSWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGUYDOMJUG44DOMQ . You are receiving this because you are subscribed to this thread.Message ID: @.***>

piyanatk commented 1 week ago

@mariogrs I think plotting the median is a good idea for the first plot. For the zoom in, we don't necessarily need to see the shape of the HRS track right since it is already appear in the first plot. Instead of that, I think it will be more useful to plot the rms around the median as a function of time for the scan-only tracks.