cosanlab / neighbors

A package to perform collaborative filtering on emotion datasets.
https://cosanlab.github.io/neighbors
MIT License
10 stars 8 forks source link

Convolution position. #41

Open ljchang opened 9 months ago

ljchang commented 9 months ago

We should verify that we are actually centering the kernel convolution as is mentioned in the documentation, or if instead we are doing a forward convolution (which is what we are likely doing). This is a very small potential bug that I doubt will amount to any appreciable difference, but would good to ensure that we are doing what we say.