models icon indicating copy to clipboard operation
models copied to clipboard

add 1st pytorch example

Open radekosmulski opened this issue 2 years ago • 2 comments

This adds the first pytorch example 🥳

Additionally, I propose in DLRMModel we rename dim to embedding_dim. This aligns us with the tf api and (which is probably more important) is more informative to the reader (this is the language used in the paper, otherwise it is a bit confusing what does output refer to -- is it the output of the model? of mlp hidden layers? embedding_dim is more informative)

radekosmulski avatar Jul 05 '23 02:07 radekosmulski

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB