it makes sense totally. I think we have to rename StringFeatures to VariableLengthVectors to avoid confusion?