Skip to content

DeepFm应该是实现错了 #11

@pbbigchicken

Description

@pbbigchicken

DeepFm里面的FM层应该把self.v参数去掉才对,这里面相当于对one-hot之后的embedding又做了一次embedding,换句话说这里不应该用原始完整的FM层。因为外面的embedding已经实现了FM里面的交叉矩阵。

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions