Closed benofben closed 1 year ago
The mathy word is that we want a inductive embedding. Otherwise we're doing some things with leakage between train/test that we shouldn't. Need to change from Fast RP. We'd orginially used it because other embeddings weren't available.
Closing as we're not doing embeddings anymore.
Still need to look into this.