Details
-
Type:
Bug
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: meas_base
-
Labels:None
-
Story Points:1
-
Epic Link:
-
Sprint:Science Pipelines DM-S15-6
-
Team:Data Release Production
Description
Serge Monkewitz has discovered that our schema aliases for even the default configuration of measurement algorithms involve cycles, because the slot centroid algorithm contains a reference to its own flag. Fixing this should just involve an extra check in SafeCentroidExtractor.
It turns out we already had code that was supposed to fix this (by resolving one alias explicitly when defining another that depended on it), but it was subverted by the fact that the slot aliases weren't being defined until after that code ran. With that fixed, I think we may need to get rid of the need for multi-level alias resolution entirely, which would be a big simplification. I'll explore that idea on DM-3401.