spyglass icon indicating copy to clipboard operation
spyglass copied to clipboard

Allow `populate` with secondary keys

Open CBroz1 opened this issue 5 months ago • 1 comments

Currently, we use the passed attrs of populate as a restriction with the dj func _jobs_to_do

https://github.com/LorenFrankLab/spyglass/blob/master/src/spyglass/utils/dj_mixin.py#L597-L600

This relies on it being a valid restriction of key_source, which only includes primary keys. In a future enhancement, we would allow secondary keys of parents in this field. The need for primary keys is perhaps not transparent in documentation

CBroz1 avatar Sep 11 '24 18:09 CBroz1