PaddlePaddle / Paddle

PArallel Distributed Deep LEarning: Machine Learning Framework from Industrial Practice (『飞桨』核心框架,深度学习&机器学习高性能单机、分布式训练和跨平台部署)
http://www.paddlepaddle.org/
Apache License 2.0
22.24k stars 5.58k forks source link

build error uint64_t undefined in alloc_continuous_space_for_grad_pass.h #17584

Closed jnorwood closed 5 years ago

jnorwood commented 5 years ago

为使您的问题得到快速解决,在建立Issues前,请您先通过如下方式搜索是否有相似问题:【搜索issue关键字】【使用labels筛选】【官方文档】

如果您没有查询到相似问题,为快速解决您的提问,建立issue时请提供如下细节信息:

Thank you for contributing to PaddlePaddle. Before submitting the issue, you could search issue in the github in case that there was a similar issue submitted or resolved before. If there is no solution,please provide us with the following details : System information -PaddlePaddle version (eg.1.1)or CommitID -CPU: including CPUMKL/OpenBlas/MKLDNN version -GPU: including CUDA/cuDNN version -OS Platform and Distribution(eg.Mac OS 10.14) -Python version To Reproduce Steps to reproduce the behavior Describe your current behavior Code to reproduce the issue Other info / logs

jnorwood commented 5 years ago

latest build from source, using ubuntu 18.04, gcc7.4.0 ` git clone https://github.com/PaddlePaddle/Paddle.git

mkdir Paddledbg

cd Paddledbg

cmake -DFLUID_INFERENCE_INSTALL_DIR=/home/jay/PaddleInst \
      -DCMAKE_BUILD_TYPE=RELWITHDEBINFO \
      -DWITH_PYTHON=OFF \
      -DWITH_MKL=OFF \
      -DWITH_TESTING=ON \
      -DWITH_GPU=OFF  \
      -DON_INFER=ON \
      ../Paddle

home/jay/Paddle/paddle/fluid/framework/ir/alloc_continuous_space_for_grad_pass.h:25:1: error: ‘uint64_t’ does not name a type; did you mean ‘u_int64_t’? uint64_t GetFuseParameterMemorySize();

Fixed my own buld by including boost::cstdint, as done in other files: `

include <boost/cstdint.hpp>

void SetFuseParameterMemorySize(boost::uint64_t memory_size);

boost::uint64_t GetFuseParameterMemorySize(); `

JiabinYang commented 5 years ago

Looks like you are using a wrong version of gcc to compile paddle, plz using gcc 4.8 or 5.4 as your complier because there is some compiler diff will cause problem

jnorwood commented 5 years ago

while the build completed, several tests failed. I haven't debugged to see if it is related.

`88% tests passed, 12 tests failed out of 103

Label Time Summary: RUN_TYPE=DIST = 0.26 secproc (11 tests) RUN_TYPE=EXCLUSIVE = 0.32 secproc (1 test)

Total Test time (real) = 25.82 sec

The following tests FAILED: 37 - test_naive_executor (Failed) 92 - test_api_impl (Failed) 94 - test_inference_image_classification_vgg (Failed) 95 - test_inference_image_classification_resnet (Failed) 96 - test_inference_label_semantic_roles (Failed) 97 - test_inference_recognize_digits_mlp (Failed) 98 - test_inference_recognize_digits_conv (Failed) 99 - test_inference_recommender_system (Failed) 100 - test_inference_word2vec (Failed) 101 - test_inference_nlp (Failed) 102 - test_train_recognize_digits_mlp (Failed) 103 - test_train_recognize_digits_conv (Failed) Errors while running CTest `

JiabinYang commented 5 years ago

Could u plz post the complete test error log?

jnorwood commented 5 years ago

paddle_logs.zip I'm attaching the zip archive that contains the ctest --verbose log and the original build log, which contained the uint64_t error. I didn't save the build log after adding the boost:cstdint include, but it built and linked all the tests without error. There are a number of compiler warnings in the build log which I didn't try to correct, though.

The ctest log shows all errors except one are trying to access something under a python subdirectory in the build path. However, I built with -DWITH_PYTHON=OFF, as suggested by the documentation instructions. Maybe these tests just need to be disabled in the cmake config if WITH_PYTHON==OFF.

There is one other failure which is not doing a python subdirectory access, but I haven't tried to examine it further.

jnorwood commented 5 years ago

paddle_log2.zip

I'm attaching a build log after adding the fix. It shows the completed build, with the test cases linking successfully.

I pushed my change into an onnxtst branch in my fork of Paddle https://github.com/jnorwood/Paddle/commit/37a60f464e35bfa9a9b4d289c506fca5164d5b9b

JiabinYang commented 5 years ago

Thx for ur support for Paddle!