[src] Fix bug in resampling checks for online features (#3639)

The --allow-upsample and --allow-downsample options were not handled correctly in the code that handles resampling for computing online features.
kaldi-asr · Oct 5, 2019 · b5f4cfc · b5f4cfc
1 parent 526ade1
commit b5f4cfc
Showing 1 changed file with 3 additions and 3 deletions.
diff --git 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));