Commit d8f54c52 authored by Jan Kiene's avatar Jan Kiene
Browse files

enable rtpdump tests for float ref again

parent ffe07fa4

main-basop.yml

+0 −20
Original line number Diff line number Diff line
@@ -293,8 +293,6 @@ workflow: 
    -   BUILD_WITH_DEBUG_MODE_INFO="true"

    - fi



    - export PYTEST_ADDOPTS="${PYTEST_ADDOPTS} -k 'not rtpdump'"



    - INV_LEVEL_SCALING=$(awk "BEGIN {print 1.0 / $LEVEL_SCALING}")

    - comp_args="--mld --ssnr --odg --scalefac $INV_LEVEL_SCALING"

    - summary_args="MLD DIFF SSNR ODG"
@@ -434,9 +432,6 @@ workflow: 
    -   bash "${CI_PROJECT_DIR}"/ivas-codec-ci/snippets/basop/apply-testv-scaling.sh

    - fi



    # TODO: remove once ref branch is switched

    - export PYTEST_ADDOPTS="${PYTEST_ADDOPTS} -k 'not rtpdump'"



    # build merge target branch and correpsonding float reference

    - bash "${CI_PROJECT_DIR}"/ivas-codec-ci/snippets/basop/build-binaries.sh float-ref

    - source "${CI_PROJECT_DIR}"/ivas-codec-ci/snippets/basop/build-merge-target-binaries.sh
@@ -560,9 +555,6 @@ workflow: 
    -   bash "${CI_PROJECT_DIR}"/ivas-codec-ci/snippets/basop/apply-testv-scaling.sh

    - fi



    # TODO: remove once ref branch is switched

    - export PYTEST_ADDOPTS="${PYTEST_ADDOPTS} -k 'not rtpdump'"



    # check MR title for flag that allows regressions to be mergable

    - echo $CI_MERGE_REQUEST_TITLE > tmp.txt

    - allow_regressions_flag=$(grep -c --ignore-case "\[allow[ -]*regression\]" tmp.txt) || true
@@ -1363,10 +1355,6 @@ ivas-pytest-on-merge-request: 
    -   export PYTEST_ADDOPTS="${PYTEST_ADDOPTS} -k 'not model'"

    - fi



    - if [ "$CI_MERGE_REQUEST_TARGET_BRANCH_NAME" == "ivas-float-update" ]; then

    -   export PYTEST_ADDOPTS="${PYTEST_ADDOPTS} -k 'not rtpdump'"

    - fi



    ### prepare pytest

    # create references

    - python3 -m pytest "$TESTS_DIR_CODEC_BE_ON_MR"/"$PYTEST_SCRIPT" -v --update_ref 1
@@ -1575,10 +1563,6 @@ ivas-interop-on-merge-request: 
    - source "${CI_PROJECT_DIR}"/ivas-codec-ci/snippets/merge-request-comparison-setup-codec.sh

    - bash "${CI_PROJECT_DIR}"/ivas-codec-ci/snippets/basop/merge-request-print-pinned-commits.sh



    - if [ "$CI_MERGE_REQUEST_TARGET_BRANCH_NAME" == "ivas-float-update" ]; then

    -   export PYTEST_ADDOPTS="${PYTEST_ADDOPTS} -k 'not rtpdump'"

    - fi



    # some helper variables - "|| true" to prevent failures from grep not finding anything

    # write to temporary file as workaround for failures observed with piping echo

    - echo $CI_MERGE_REQUEST_TITLE > tmp.txt
@@ -1811,10 +1795,6 @@ renderer-usan: 
    -   export PYTEST_ADDOPTS="${PYTEST_ADDOPTS} -k 'not model'"

    - fi



    - if [ "$CI_MERGE_REQUEST_TARGET_BRANCH_NAME" == "ivas-float-update" ]; then

    -   export PYTEST_ADDOPTS="${PYTEST_ADDOPTS} -k 'not rtpdump'"

    - fi



    - make_args="CLANG=$CLANG_NUM"

    # for MRs to main, the whole sanitizer test takes too long (even with stv set of inputs)

    # shorten files to 1s for that case