-
Notifications
You must be signed in to change notification settings - Fork 5.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Pglbox2.0] merge gpugraph to develop #49946
Conversation
* add set slot_num_for_pull_feature for psgpuwarper
* add get_epoch_finish interface * add return * delete return
* fix miss key for error dataset * fix miss key for error dataset Co-authored-by: yangjunchao <yangjunchao@baidu.com>
Co-authored-by: yangjunchao <yangjunchao@baidu.com>
* check node&feature format when loading * check node&feature format when loading (2£ (2)
你的PR提交成功,感谢你对开源项目的贡献! |
❌ The PR is not created using PR's template. You can refer to this Demo. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for unity_build_rule.cmake
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
GTM
# limitations under the License. | ||
|
||
from paddle.fluid.data_feeder import check_variable_and_dtype | ||
from paddle.fluid.layer_helper import LayerHelper |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fluid API再被移除,这里是否一定需要引用 paddle.fluid
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个incubate目录下的其余API目前也仍然都在使用FLUID.得后续统一迁移,这个PR暂时先不修改了
|
||
Args: | ||
input (Variable): The zipped input, 2-D LodTensor with shape [N, M]. | ||
lod (Variable): The original lod of unzipped input, 1-D LodTensor with shape[K]. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
对外已经不在提LoDTensor、LoD概念,这里docs和参数命名需要依据含义再改下
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个OP是内部业务使用,主要就是根据LOD信息来操作INPUT,不打算暴露,这里的LodTensor的说法,可以再提个pr修改,暂时这个pr先豁免
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approve for API
PR types
New features
PR changes
Others
Describe
日志降级
新增paddle.incubate.operators.unzip
修复dense参数卡住问题
增加 get_epoch_finish python端接口
修复barrier不生效的问题