From 9798e3af047da80b8904d2955dbb2bb099b18fa4 Mon Sep 17 00:00:00 2001
From: Ewald Enzinger <ewald.enzinger@entn.at>
Date: Fri, 4 Oct 2019 18:05:18 +0200
Subject: [PATCH] Fix bug in resampling checks for online features

The --allow-upsample and --allow-downsample options were not handled correctly in the code that handles resampling for computing online features.
---
 src/feat/online-feature.cc | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/feat/online-feature.cc b/src/feat/online-feature.cc
index 6f5ce6ee95b..047909e7a21 100644
--- a/src/feat/online-feature.cc
+++ b/src/feat/online-feature.cc
@@ -88,10 +88,10 @@ void OnlineGenericBaseFeature<C>::MaybeCreateResampler(
   if (resampler_ != nullptr) {
     KALDI_ASSERT(resampler_->GetInputSamplingRate() == sampling_rate);
     KALDI_ASSERT(resampler_->GetOutputSamplingRate() == expected_sampling_rate);
-  } else if (((sampling_rate > expected_sampling_rate) &&
-              !computer_.GetFrameOptions().allow_downsample) ||
+  } else if (((sampling_rate < expected_sampling_rate) &&
+              computer_.GetFrameOptions().allow_downsample) ||
              ((sampling_rate > expected_sampling_rate) &&
-              !computer_.GetFrameOptions().allow_upsample)) {
+              computer_.GetFrameOptions().allow_upsample)) {
     resampler_.reset(new LinearResample(
         sampling_rate, expected_sampling_rate,
         std::min(sampling_rate / 2, expected_sampling_rate / 2), 6));