search

HERA-Team / hera_corr_f

HERA F-Engine on SNAP
1 stars 5 forks source link

Make redis values redis-accepted types #21

Closed mkolopanis closed 4 years ago

mkolopanis commented 4 years ago

Newer redis verisons are much stricter on types that are allowed to be put into the database. I have found some list, bool, and Nonetype entries trying to sneak in in hera_snap_redis_monitor.py and have tried to add a block to be robust against these types.

Current check looks like things get plumbed through enough on the corr_cm side. I tried to not change anything in a way that would make corr_cm unhappy.

Still maybe need a full testing run, but have received some data.

image

mkolopanis commented 4 years ago

Looks to be working well. Still requires a daemon restart occasionally but I think there is a deeper issue that will require research.