From 723200fabd4a6e40f9d5feeea2baf3dc25710473 Mon Sep 17 00:00:00 2001 From: Aditya Ujeniya Date: Wed, 4 Dec 2024 17:53:18 +0100 Subject: [PATCH] fix: SimpleResampler fixed --- .gitignore | 2 +- pkg/resampler/resampler.go | 10 +++++----- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/.gitignore b/.gitignore index 87a8070..f2ce56d 100644 --- a/.gitignore +++ b/.gitignore @@ -20,7 +20,7 @@ aditya.creds test.creds -config.json +/config.json migrateTimestamps.pl test_ccms_api.sh diff --git a/pkg/resampler/resampler.go b/pkg/resampler/resampler.go index 5377f18..c641670 100644 --- a/pkg/resampler/resampler.go +++ b/pkg/resampler/resampler.go @@ -8,20 +8,20 @@ import ( "github.com/ClusterCockpit/cc-metric-store/internal/util" ) -func SimpleResampler(data []util.Float, old_frequency int64, new_frequency int64) ([]util.Float, error) { +func SimpleResampler(data []util.Float, old_frequency int64, new_frequency int64) ([]util.Float, int64, error) { if old_frequency == 0 || new_frequency == 0 || new_frequency <= old_frequency { - return nil, errors.New("either old or new frequency is set to 0") + return data, old_frequency, nil } if new_frequency%old_frequency != 0 { - return nil, errors.New("new sampling frequency should be multiple of the old frequency") + return nil, 0, errors.New("new sampling frequency should be multiple of the old frequency") } var step int = int(new_frequency / old_frequency) var new_data_length = len(data) / step if new_data_length == 0 || len(data) < 100 || new_data_length >= len(data) { - return data, nil + return data, old_frequency, nil } new_data := make([]util.Float, new_data_length) @@ -30,7 +30,7 @@ func SimpleResampler(data []util.Float, old_frequency int64, new_frequency int64 new_data[i] = data[i*step] } - return new_data, nil + return new_data, new_frequency, nil } // Inspired by one of the algorithms from https://skemman.is/bitstream/1946/15343/3/SS_MSthesis.pdf