[ ] Call to enumerate should be replaced by the actual col number (without assuming they are zero indexed) and in turn converted to the corresponding loom rank.
[ ] When populating the cell values, target columns should be ignored.
[ ] Should check that user constraints do not collide with the user provided constraints.
fsaad
commented
6 years ago
The problematic code: https://github.com/probcomp/bayeslite/blob/c49fb01cbd918f565aeb0a00669434c6bf16dbeb/src/backends/loom_backend.py#L672-L685
enumerateshould be replaced by the actual col number (without assuming they are zero indexed) and in turn converted to the corresponding loom rank.Likely related: #590.