Oneflow-Inc / oneflow-api-cn

Chinese Documents of OneFlow API
https://start.oneflow.org/oneflow-api-cn/
7 stars 3 forks source link

one_embedding 翻译 #75

Closed robin978 closed 2 years ago

robin978 commented 2 years ago

oneflow.one_embedding.MultiTableEmbedding

截屏2022-05-11 下午4 10 36

oneflow.one_embedding.save_snapshot oneflow.one_embedding.load_snapshot

截屏2022-05-11 下午4 10 59

oneflow.one_embedding.forward

截屏2022-05-11 下午4 11 16
guo-ran commented 2 years ago

另外最新的代码里做了一点更新,新增了个接口,可以加上吗

rainyq69 commented 2 years ago

另外最新的代码里做了一点更新,新增了个接口,可以加上吗

可以的,我稍后加上。 另外麻烦看一下这些可以吗:https://github.com/Oneflow-Inc/oneflow-api-cn/pull/75#discussion_r853731544https://github.com/Oneflow-Inc/oneflow-api-cn/pull/75#discussion_r853734938https://github.com/Oneflow-Inc/oneflow-api-cn/pull/75#discussion_r853736049

rainyq69 commented 2 years ago

oneflow.one_embedding.MultiTableMultiColumnEmbedding

截屏2022-05-11 下午4 13 36
rainyq69 commented 2 years ago

基类 Embedding 中的 forward 函数的 docstring 描述是否有问题?因为 MultiTableEmbedding 和 MultiTableMultiColumnEmbedding 都继承自它,但这个 docstring 的描述只提及了前者。

这导致了目前文档中的问题: 图片

guo-ran commented 2 years ago

基类 Embedding 中的 forward 函数的 docstring 描述是否有问题?因为 MultiTableEmbedding 和 MultiTableMultiColumnEmbedding 都继承自它,但这个 docstring 的描述只提及了前者。

这导致了目前文档中的问题: 图片

确实是的,原始文档中Embedding对forward的描述不准确,是不是去掉前向计算这行描述?或者改成Embedding前向计算?

rainyq69 commented 2 years ago

是不是去掉前向计算这行描述?或者改成Embedding前向计算?

我觉得可以改成 Embedding(基类)及其子类的前向计算。

robin978 commented 2 years ago

oneflow.one_embedding.make_device_mem_store_options

截屏2022-05-11 下午4 16 24

oneflow.one_embedding.make_cached_ssd_store_options

截屏2022-05-11 下午4 17 35

oneflow.one_embedding.make_cached_host_mem_store_options

截屏2022-05-11 下午4 19 39

oneflow.one_embedding.make_uniform_initializer

截屏2022-05-11 下午4 21 12

oneflow.one_embedding.make_normal_initializer oneflow.one_embedding.make_table_options

截屏2022-05-11 下午4 23 33
robin978 commented 2 years ago

oneflow.one_embedding.Ftrl

截屏2022-05-11 下午4 30 38
MARD1NO commented 2 years ago

oneflow.one_embedding.Ftrl 截屏2022-05-11 下午4 30 38

我的问题,这个应该是

ftrl = flow.one_embedding.FTRL

我提交个PR修改下。。