Skip to content

Commit

Permalink
Merge pull request #1140 from hannahxy13/feature/validation_steps
Browse files Browse the repository at this point in the history
Updated run-docker.sh to include values needed for verification
  • Loading branch information
auphelia authored Jul 25, 2024
2 parents 2747853 + 65a356a commit 912cadf
Showing 1 changed file with 30 additions and 0 deletions.
30 changes: 30 additions & 0 deletions run-docker.sh
Original file line number Diff line number Diff line change
Expand Up @@ -265,6 +265,36 @@ if [ ! -z "$FINN_XILINX_PATH" ];then
DOCKER_EXEC+="-e ALVEO_TARGET_DIR=$ALVEO_TARGET_DIR "
fi
fi

# This part is used for internal ci for finn-examples
# if using build verification for finn-examples ci, set up the necessary Docker variables
if [ "$VERIFICATION_EN" = 1 ]; then
if [ -z "$FINN_EXAMPLES_ROOT" ]; then
recho "FINN_EXAMPLES_ROOT path has not been set."
recho "Please set FINN_EXAMPLES_ROOT path to enable verification."
exit -1
elif [ ! -d "${FINN_EXAMPLES_ROOT}/ci" ]; then
recho "ci folder not found in ${FINN_EXAMPLES_ROOT}."
recho "Please ensure the FINN-examples repo has been set up correctly, and FINN_EXAMPLES_ROOT path is set correctly, to enable verification."
exit -1
elif [ -z "$VERIFICATION_IO" ]; then
recho "VERIFICATION_IO paths has not been set."
recho "Please ensure the path to the input and expected output files has been set correctly to eneable verification."
exit -1
elif [ ! -d "$VERIFICATION_IO" ]; then
recho "${VERIFICATION_IO} is not a directory."
recho "Please ensure the VERIFICATION_IO path has been set to the directory containing the input and expected output files for verification."
exit -1
else
DOCKER_EXEC+="-e VERIFICATION_EN=$VERIFICATION_EN "
DOCKER_EXEC+="-e FINN_EXAMPLES_ROOT=$FINN_EXAMPLES_ROOT "
DOCKER_EXEC+="-e VERIFICATION_IO=$VERIFICATION_IO "
FINN_DOCKER_EXTRA+="-v $FINN_EXAMPLES_ROOT/ci:$FINN_EXAMPLES_ROOT/ci "
FINN_DOCKER_EXTRA+="-v $VERIFICATION_IO:$VERIFICATION_IO "
fi
fi


DOCKER_EXEC+="$FINN_DOCKER_EXTRA "

if [ -z "$FINN_SINGULARITY" ];then
Expand Down

0 comments on commit 912cadf

Please sign in to comment.