[feature] new img container support PNG #380
Conversation
|
Because of the new dependencies and changes to the cmake file, a new version README is needed to guide users build with new container |
The build commands for the examples has not changed, but the build command for test has changed |
when i build buddy-mlir, i have to enable BUDDY_MLIR_ENABLE_DIP_LIB, otherwise, the build won't succeed. It's best to explain this in the pr. |
Thank you for your question. I have updated the PR to prevent compilation failure due to unspecified parameters |
|
Does libpng have specific version requirements? My current version is 1.6.37 when i use new container in MobileNet example, I meet this error
Can you try the MobileNet example on your side ? Thank you ! |
Please refer to the Lenet's CMakeLists.txt for the correct link to the libpng library |
| @@ -57,5 +57,5 @@ SET_TARGET_PROPERTIES(LENET PROPERTIES LINKER_LANGUAGE C) | |||
| add_executable(buddy-lenet-run buddy-lenet-main.cpp) | |||
| target_link_directories(buddy-lenet-run PRIVATE ${LLVM_LIBRARY_DIR}) | |||
|
|
|||
| set(BUDDY_LENET_LIBS LENET mlir_c_runner_utils ${OpenCV_LIBS}) | |||
| set(BUDDY_LENET_LIBS LENET mlir_c_runner_utils ${OpenCV_LIBS} ${PNG_LIBRARY}) | |||
| this->sizes[3] = this->width; | ||
| this->setStrides(); | ||
| size_t size = this->product(this->sizes); | ||
| std::cout << "size" << size << std::endl; |
New img container support PNG based on libpng
To compile target buddy-new-image-container-test:
$ cd buddy-mlir
$ mkdir build
$ cd build
$ cmake -G Ninja ..
-DMLIR_DIR=$PWD/../llvm/build/lib/cmake/mlir
-DLLVM_DIR=$PWD/../llvm/build/lib/cmake/llvm
-DLLVM_ENABLE_ASSERTIONS=ON
-DCMAKE_BUILD_TYPE=RELEASE
-DBUDDY_MLIR_ENABLE_DIP_LIB=ON
$ ninja