高璟琦
高璟琦
groupBy~ i'am sorry it takes so long to log back on github
@Wicknight Thanks for your kind advice. I have taken them into account and committed to a new version. Plz, check it.
@Wicknight Thanks for your advice again. Now I just tested the FISSA and SASRec models with all default settings on the ml-100k dataset, which means they use almost all similar...
@Wicknight Thanks for your kind comments. I fixed them according to your advice. It makes sense!
@Wicknight Thank you for your comments. Is there anything need to be improved?
@Wicknight Thanks for your concern. According to the paper, they adopt binary cross-entropy. Whereas we usually use CE and BPR in recbole for most sequential models. I also think it's...
> > @Wicknight Thanks for your concern. According to the paper, they adopt binary cross-entropy. Whereas we usually use CE and BPR in recbole for most sequential models. I also...
> > > > @Wicknight Thanks for your concern. According to the paper, they adopt binary cross-entropy. Whereas we usually use CE and BPR in recbole for most sequential models....
> Yes, did you test on them?How about performance gap?
> > > > > > > > > Yes, did you test on them?How about performance gap? > > First of all, thank you for your PR and efforts...