Vitis-Tutorials icon indicating copy to clipboard operation
Vitis-Tutorials copied to clipboard

makefile and readme needs modification

Open vivianyin opened this issue 3 years ago • 1 comments

https://github.com/Xilinx/Vitis-Tutorials/blob/master/AI_Engine_Development/Design_Tutorials/01-aie_lenet_tutorial/Makefiles/Makefile

Line 63, SYSROOT_PATH needs to be changed with the path to xilinx-versal-common-v2020.2 at user's end. This should be added to Readme.

Line 247, --package.sd_dir $(PLATFORM_REPO_PATHS)/sw/versal/xrt This line should be removed?

Line 248-249, the paths need to be changed with the path to xilinx-versal-common-v2020.2 at user's end. This should be added to Readme.

vivianyin avatar Apr 15 '21 07:04 vivianyin

@ThomasXilinx can you have someone look at this?

Premduth avatar Aug 18 '21 16:08 Premduth

Hi @SURUTHI1605 , I think this lenet issue has been solved. Please confirm.

imrickysu avatar Jan 13 '23 08:01 imrickysu

I'm unable to access this link https://github.com/Xilinx/Vitis-Tutorials/blob/master/AI_Engine_Development/Design_Tutorials/01-aie_lenet_tutorial/Makefiles/Makefile

SURUTHI1605 avatar Feb 09 '23 06:02 SURUTHI1605

In 2022.2 branch, the makefile path is now https://github.com/Xilinx/Vitis-Tutorials/blob/2022.2/AI_Engine_Development/Design_Tutorials/01-aie_lenet_tutorial/Makefile .

Please fix in the latest branch.

Please document the environment variables need to be set before running the makefile, e.g. $COMMON_IMAGE_VERSAL on top of the Makefile and README, and provide proper error messages if these variables are not set.

imrickysu avatar Feb 09 '23 06:02 imrickysu

The environment variables need to be set was separately defined in sample_env_setup.sh script and the steps to reproduce the script was documented in README.

SURUTHI1605 avatar May 07 '23 15:05 SURUTHI1605