diff options
author | Jeremy Johnson <jeremy.johnson@arm.com> | 2024-02-19 13:57:21 +0000 |
---|---|---|
committer | Eric Kunze <eric.kunze@arm.com> | 2024-02-26 16:20:32 +0000 |
commit | 08965d35f728d93d8b215753b4b270a422fe39c9 (patch) | |
tree | 41a520e5f83d3c5bfa9db559bfc500753ba95bb4 /reference_model/src/verify/verify_relative.cc | |
parent | e4c89eb171272c1bc28c188aeb8363009674ba86 (diff) | |
download | reference_model-08965d35f728d93d8b215753b4b270a422fe39c9.tar.gz |
Verifier - change to output largest error deviance
Add general validateData function used by ABS_ERROR, ULP,
RELATIVE and REDUCE_PRODUCT to find and output largest deviance
from the error bounds.
Clean up naming inconsistencies bewteen verify modes.
Signed-off-by: Jeremy Johnson <jeremy.johnson@arm.com>
Change-Id: Ib903faf36f784cacae91edab61d8e489461a727c
Diffstat (limited to 'reference_model/src/verify/verify_relative.cc')
-rw-r--r-- | reference_model/src/verify/verify_relative.cc | 30 |
1 files changed, 9 insertions, 21 deletions
diff --git a/reference_model/src/verify/verify_relative.cc b/reference_model/src/verify/verify_relative.cc index b12daf7..5085fd9 100644 --- a/reference_model/src/verify/verify_relative.cc +++ b/reference_model/src/verify/verify_relative.cc @@ -24,27 +24,13 @@ namespace TosaReference namespace { -template <typename OutDtype> -bool validateData(const double* ref, - const OutDtype* imp, - const std::vector<int32_t>& shape, - const RelativeVerifyInfo& cfg) +double calcErrorBound(double referenceValue, double boundsValue, const void* cfgPtr) { - const size_t T = static_cast<size_t>(numElements(shape)); - TOSA_REF_REQUIRE(T > 0, "[R] Invalid shape for reference tensor"); + const auto cfg = reinterpret_cast<const RelativeVerifyInfo*>(cfgPtr); + unused(referenceValue); + unused(boundsValue); - double errBound = cfg.max * cfg.scale; - for (size_t i = 0; i < T; ++i) - { - bool valid = tosaCheckFloatBound(imp[i], ref[i], errBound); - if (!valid) - { - auto pos = indexToPosition(i, shape); - WARNING("[Verifier][RP] Location %s", positionToString(pos).c_str()); - return false; - } - } - return true; + return cfg->max * cfg->scale; } } // namespace @@ -61,17 +47,19 @@ bool verifyRelative(const CTensor* referenceTensor, const double* refData = reinterpret_cast<const double*>(referenceTensor->data); TOSA_REF_REQUIRE(refData != nullptr, "[R] Missing data for reference"); + const std::string modeStr = "R"; + switch (implementationTensor->data_type) { case tosa_datatype_fp32_t: { const auto* impData = reinterpret_cast<const float*>(implementationTensor->data); TOSA_REF_REQUIRE(impData != nullptr, "[R] Missing data for implementation"); - return validateData(refData, impData, refShape, rInfo); + return validateData(refData, nullptr, impData, refShape, modeStr, &rInfo, &calcErrorBound); } case tosa_datatype_fp16_t: { const auto* impData = reinterpret_cast<const half_float::half*>(implementationTensor->data); TOSA_REF_REQUIRE(impData != nullptr, "[R] Missing data for implementation"); - return validateData(refData, impData, refShape, rInfo); + return validateData(refData, nullptr, impData, refShape, modeStr, &rInfo, &calcErrorBound); } default: WARNING("[Verifier][R] Data-type not supported."); |