spyglass icon indicating copy to clipboard operation
spyglass copied to clipboard

`DLCModelSource` uses `fetch1` when restricting by a secondary key

Open CBroz1 opened this issue 5 months ago • 0 comments

This portion of position_dlc_model

  1. Uses a FreeTable to fetch a dependency
  2. Presumes uniqueness of project_name that is not enforced by the DLCModelTraining table

https://github.com/LorenFrankLab/spyglass/blob/a7b13fa1636f0b219fc7d640e25f8269f41ac93e/src/spyglass/position/v1/position_dlc_model.py#L96-L99

These should be removed in favor if restricting by the primary key of the parent, used directly

CBroz1 avatar Sep 11 '24 22:09 CBroz1