fix!: Fix wrong directional check for low is worse metrics #3698

Job	Run time
C++ Test Coverage (gets coverage of main branch, currently not main branch because no C++ on main)	24m 32s
Build + Test (C++) (Release, gcc, g++, 11)	17m 47s
format	7s
Build + Test (C++) (Release, clang, clang++, 14)	21m 37s
Mypy Type Checks	2m 10s
Pre-commit Checks	27s
Build + Test (C++) (Release, clang, clang++, 17, true)	30m 23s
Build + Test (C++) (Debug, gcc, g++, 11)	18m 9s
pylint	2m 47s
Build + Test (C++) (Debug, gcc, g++, 12)	21m 27s
tidy	20m 3s
unittests	11m 6s
Build + Test (C++) (Debug, clang, clang++, 14)	20m 25s
Build + Test (C++) (Debug, clang, clang++, 17, true)	27m 18s
Build + Test (C++) (clang, clang++, 14, Tsan)	23m 14s
Build + Test (C++) (clang, clang++, 14, Asan)	29m 14s
integrationtests-asan	1h 4m 20s
integrationtests-debug	1h 2m 53s
dockerized-unittests	14m 24s
Comment Coverage Results	5s
integrationtests-tsan	1h 20m 1s
integrationtests-release	1h 5m 46s
	9h 18m 15s

Provide feedback