search

Johnson0722 / CTR_Prediction

CTR prediction using FM FFM and DeepFM
745 stars 298 forks source link

self.X also need tf.pow(self.X, 2) #8

Open lsq357 opened 6 years ago

lsq357 commented 6 years ago

https://github.com/Johnson0722/CTR_Prediction/blob/ec4de3d2654c5d808ff633d9a10cdd04d636db1e/FM/FM.py#L58

zouguojian commented 5 years ago

right, I have seen this wrong yet

zouguojian commented 5 years ago

tf.reduce_mean is wrong ,should to change as tf.reduce_sum

JobQiu commented 4 years ago

https://github.com/Johnson0722/CTR_Prediction/blob/ec4de3d2654c5d808ff633d9a10cdd04d636db1e/FM/FM.py#L58 tf.reduce_mean is wrong ,should to change as tf.reduce_sum

is this the equation?