Do not merge nor copy monitoring tables when merging DL1 h5 files

cta-observatory / cta-lstchain

LST prototype testbench chain

https://cta-observatory.github.io/cta-lstchain/

BSD 3-Clause "New" or "Revised" License

24 stars 77 forks source link

Do not merge nor copy monitoring tables when merging DL1 h5 files #1256

Closed morcuended closed 4 months ago

morcuended commented 4 months ago

closes #1193

FrancaCassol commented 4 months ago

Hi, agree with the change, for the moment I would just remove the check of the length. In a short future the second line of (dl1_params_tel_mon_ped_key, dl1_params_tel_mon_cal_key, dl1_params_tel_mon_flat_key) will be in anyway suppressed since we are going to use (dl1_mon_tel_CatB_cal_key, dl1_mon_tel_CatB_ped_key, dl1_mon_tel_CatB_flat_key)

morcuended commented 4 months ago

Can you make that they are neither merged nor copied?

I brought back the check of the same length, but I removed the monitoring tables from the keys to be copied list. Also, remove those keys from the ones to be merged as suggested by @moralejo.