fit_kit 0.2.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6f5d819105551844f5a7e45a8399aa60b403cbc5b8b2adb278c0fb9a8a29631c
-  data.tar.gz: 1a55ae0af1a09860935dc3dee596a191c42133ae2a070c7c788514e4b4694248
+  metadata.gz: 10c537aabfcdc0d4542be231629868eca99e14a6f5ee2cf7a3f313e964f14ddc
+  data.tar.gz: 218b078edf26e38088d3da9565be4243ea83a1d5eb56d8f570e600795bd21a3f
 SHA512:
-  metadata.gz: 46cd84be49270a1197f815f776cc51627ba58d7abf72d8f0f4b6d80cb9f485ceca1a73745eb62df2696fe2a0cd3b78e2311550bf3f711dfcaf133daeed3abf1e
-  data.tar.gz: 32993953f6611a2ad85a70426be7525bb558e3cdf201e994e6848d04ab29f8ebdee3e1d722ac432173f05e799196f47a8f38e3d73c6f151c7de83497cdfa3463
+  metadata.gz: 648647a0900cb94c8828f48935196b3e4395206c4fb08dd122a0c4300f156a0f3a507f0ee9bf141a1456cb243adbefdb881716b42bf1793ce81e334414336e19
+  data.tar.gz: 217bc0fb924c5fe02b444252ac6c4547294bcde9de6ee913459e7fe742efa8d3fce092118272cd8c0559b29f0cbf7733b75b72fe247d01ddd9b0bc37a7a9c72d

data/CHANGELOG.md

@@ -1,4 +1,10 @@
-## [Unreleased]
+## [0.3.0] - 2024-10-16
+
+- Simplify to return hash for the parsing
+
+## [0.2.0] - 2024-10-15
+
+- Simplify to return hash fileds for records
 
 ## [0.1.0] - 2024-10-14
 

data/Cargo.lock

@@ -1,6 +1,6 @@
 # This file is automatically @generated by Cargo.
 # It is not intended for manual editing.
-version = 3
+version = 4
 
 [[package]]
 name = "aho-corasick"
@@ -136,9 +136,9 @@ dependencies = [
 
 [[package]]
 name = "fitparser"
-version = "0.7.0"
+version = "0.9.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "3c57b6c1de15f92aa2846a6286eefc115637405a4298eea11a65799aa4ce4177"
+checksum = "dc8dfc12dd3f7ec76da4609addc6700a3d5c9e37e8c0e924c7fbd61bc52d70a7"
 dependencies = [
  "chrono",
  "nom",
@@ -228,9 +228,9 @@ checksum = "a7a70ba024b9dc04c27ea2f0c0548feb474ec5c54bba33a7f72f873a39d07b24"
 
 [[package]]
 name = "magnus"
-version = "0.6.4"
+version = "0.7.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "b1597ef40aa8c36be098249e82c9a20cf7199278ac1c1a1a995eeead6a184479"
+checksum = "3d87ae53030f3a22e83879e666cb94e58a7bdf31706878a0ba48752994146dab"
 dependencies = [
  "magnus-macros",
  "rb-sys",

data/README.md

@@ -1,5 +1,7 @@
 # FitKit
 
+> Not production ready yet.
+
 `fit_kit` is a small ruby gem wraps Rust's crate [fitparse-rs](https://github.com/stadelmanma/fitparse-rs) to provide effecient and fast Garmin fit files parsing.
 
 ## Installation
@@ -17,9 +19,20 @@ If bundler is not being used to manage dependencies, install the gem by executin
 ```ruby
 test_fit_file = File.join(Dir.pwd, "example.fit")
 fit_data_records = ::FitKit.parse_fit_file(test_fit_file)
-# [RFitDataRecord, RFitDataRecord, RFitDataRecord ...]
+# { record: [{...}, {...}], session: [{...}], lap: [..], activity: [...] }
 ```
 
+## Features
+There are couple of more built-in features can simplify the consuming of the data significantly:
+
+- `records_hash`   - All records in a ruby hash format
+- `avg_for`        - Average of a given field
+- `elevation_gain` - Elevation gain. Calculate the overall elevation gain. Also allow to pass in a field.
+- `calculate_partition_indices` - Calculate the partition indices for a given field
+- `partition_stats_for_fields` - Calculate the partition stats for a given field. Useful for calculating splits data.
+- `sample_series_for_records` - Sample series for a given field, useful for plotting
+- `zone_time_for` - Heart Rate Zones data
+
 ## Performance
 Here is the performance parsing __4090__ fit files on my M1 Mac Mini (16G, 8 Cores) took 6 seconds (in parallel):
 

data/ext/fit_kit/Cargo.toml

@@ -10,5 +10,5 @@ publish = false
 crate-type = ["cdylib"]
 
 [dependencies]
-fitparser = "0.7.0"
-magnus = { version = "0.6.2" }
+fitparser = "0.9.0"
+magnus = { version = "0.7.1" }

data/ext/fit_kit/src/lib.rs

@@ -1,75 +1,524 @@
-use fitparser::{self, FitDataRecord, Value};
+use fitparser::{self, profile::MesgNum, FitDataField, FitDataRecord, Value};
 use magnus::{function, method, prelude::*, Error, IntoValue, RArray, RHash, Ruby, Symbol};
-use std::fs::File;
+use std::{
+    collections::{BTreeMap, HashMap},
+    fs::File,
+};
+
+/// Extesnion methods for FitDataRecord
+pub trait FitDataRecordExt {
+    fn timestamp(&self) -> Option<i64>;
+    fn field_value(&self, field_name: &str) -> Option<(Value, String)>;
+}
+
+impl FitDataRecordExt for FitDataRecord {
+    fn timestamp(&self) -> Option<i64> {
+        self.fields()
+            .iter()
+            .find(|field| field.name() == "timestamp")
+            .and_then(|field| match field.value() {
+                Value::Timestamp(v) => Some(v.timestamp()),
+                _ => None,
+            })
+            .map(|v| v as i64)
+    }
+
+    fn field_value(&self, field_name: &str) -> Option<(Value, String)> {
+        let field = self
+            .fields()
+            .iter()
+            .find(|field| field.name() == field_name)?;
+
+        Some((field.value().clone(), field.units().to_string()))
+    }
+}
+
+/// extension methods for vec of FitDataRecord
+pub trait FitDataRecordVecExt {
+    fn sample_series_for_records(
+        &self,
+        field_name: String,
+        num_of_points: u16,
+    ) -> Vec<(i64, f64, String)>;
+    fn aggregate_field_values(&self, records: Vec<&FitDataRecord>, field: &str) -> (f64, String);
+}
+
+impl FitDataRecordVecExt for Vec<FitDataRecord> {
+    fn sample_series_for_records(
+        &self,
+        field_name: String,
+        num_of_points: u16,
+    ) -> Vec<(i64, f64, String)> {
+        // if there are no records, return empty vec
+        if self.is_empty() {
+            return vec![];
+        }
+
+        // find the min and max timestamp
+        let min_timestamp = self
+            .iter()
+            .find(|r| r.kind() == MesgNum::Record && r.timestamp().is_some())
+            .and_then(|r| r.timestamp());
+
+        let max_timestamp = self
+            .iter()
+            .rev()
+            .find(|r| r.kind() == MesgNum::Record && r.timestamp().is_some())
+            .and_then(|r| r.timestamp());
+
+        // if both exists and min is less than max, we proceed
+        // otherwise return empty vec
+        let (min, max) = match (min_timestamp, max_timestamp) {
+            (Some(min), Some(max)) if min < max => (min, max),
+            _ => return vec![],
+        };
+
+        print!("min: {:?}, max: {:?}", min, max);
+
+        // calculate the interval we need to sample
+        let total_duration_in_seconds = max - min;
+        let interval = total_duration_in_seconds / num_of_points as i64;
+
+        // now we can group them into buckets
+        let mut sample_data = BTreeMap::new();
+        for record in self.into_iter().filter(|r| r.timestamp().is_some()) {
+            let timestamp = record.timestamp().unwrap() as i64;
+            let bucket = ((timestamp - min) / interval as i64) as u16;
+            // insert it into the bucket
+            sample_data.entry(bucket).or_insert(vec![]).push(record);
+        }
+
+        // now we can sample the data
+        sample_data
+            .into_iter()
+            .map(|(bucket, records)| {
+                let timestamp = min + (bucket as i64 * interval) as i64;
+                let (value, units) = self.aggregate_field_values(records, &field_name);
+                (timestamp, value, units)
+            })
+            .collect()
+    }
+
+    fn aggregate_field_values(&self, records: Vec<&FitDataRecord>, field: &str) -> (f64, String) {
+        // Use the new field method
+        let values: Vec<f64> = records
+            .iter()
+            .filter_map(|record| {
+                record
+                    .field_value(field)
+                    .and_then(|(v, _)| MyValue(v).as_f64())
+            })
+            .collect();
+
+        let units = records
+            .iter()
+            .find_map(|record| record.field_value(field).map(|(_, u)| u))
+            .unwrap_or_else(|| String::from(""));
+
+        // Calculate average (or use another aggregation method)
+        if values.is_empty() {
+            (0.0, String::from(""))
+        } else {
+            (values.iter().sum::<f64>() / values.len() as f64, units)
+        }
+    }
+}
+
+// wrap fitparse value
+struct MyValue(Value);
+
+impl MyValue {
+    // turn value into f64
+    fn as_f64(&self) -> Option<f64> {
+        match &self.0 {
+            Value::SInt8(i) => Some(*i as f64),
+            Value::UInt8(u) => Some(*u as f64),
+            Value::SInt16(i) => Some(*i as f64),
+            Value::UInt16(u) => Some(*u as f64),
+            Value::SInt32(i) => Some(*i as f64),
+            Value::UInt32(u) => Some(*u as f64),
+            Value::Float32(f) => Some(*f as f64),
+            Value::Float64(f) => Some(*f),
+            Value::UInt8z(u) => Some(*u as f64),
+            Value::UInt16z(u) => Some(*u as f64),
+            Value::UInt32z(u) => Some(*u as f64),
+            Value::SInt64(i) => Some(*i as f64),
+            Value::UInt64(u) => Some(*u as f64),
+            Value::UInt64z(u) => Some(*u as f64),
+            _ => None, // Handle any other variants that don't convert to f64
+        }
+    }
+}
+
+#[magnus::wrap(class = "FitParseResult")]
+struct FitParseResult(Vec<FitDataRecord>);
+
+impl FitParseResult {
+    /**
+     * Returns Ruby hash for all the records
+     * With keys are the record types
+     */
+    fn records_hash(&self) -> RHash {
+        let mut records_by_kind_vec: Vec<(String, Vec<&FitDataRecord>)> = {
+            let mut map: HashMap<String, Vec<&FitDataRecord>> = HashMap::new();
+
+            for record in self.0.iter() {
+                let kind = record.kind().to_string();
+                map.entry(kind).or_insert_with(Vec::new).push(record);
+            }
+
+            map.into_iter().collect()
+        };
+
+        // Sort the vector by the first element (kind string)
+        records_by_kind_vec.sort_by(|a, b| a.0.cmp(&b.0));
+
+        // now let's group by the record by kind
+        let result_hash = RHash::new();
+        for (kind, kind_records) in records_by_kind_vec {
+            // turn records into rarray
+            let array = RArray::new();
+            for record in kind_records {
+                // TODO here do not pass RFitDataRecord
+                // turn it into fields_hash directly
+                array.push(get_fields_hash(record)).unwrap();
+            }
+
+            result_hash.aset(Symbol::new(kind), array).unwrap();
+        }
+
+        result_hash
+    }
+
+    // summary methods
+    fn avg_for(&self, field_name: String) -> (f64, String) {
+        // passing the reference
+        self.avg_for_records(&self.0, field_name)
+    }
+
+    fn elevation_gain(&self, field_name: String) -> (f64, String) {
+        self.elevation_gain_for_records(&self.0, field_name)
+    }
+
+    // given a bunch of records, calculate the elevation gain
+    fn elevation_gain_for_records(
+        &self,
+        records: &Vec<FitDataRecord>,
+        field_name: String,
+    ) -> (f64, String) {
+        let fields: Vec<&FitDataField> = records
+            .iter()
+            .filter(|r| r.kind() == MesgNum::Record)
+            .flat_map(|r| r.fields().iter().filter(|field| field.name() == field_name))
+            .collect();
+
+        let count = fields.len();
+
+        if count == 0 {
+            return (0.0, String::from(""));
+        }
+
+        let units = fields.first().unwrap().units();
+
+        let elevation_gain_sum = fields.windows(2).fold(0.0, |acc, window| {
+            // find the field first
+            let value1 = MyValue(window[1].value().clone()).as_f64();
+            let value0 = MyValue(window[0].value().clone()).as_f64();
+
+            match (value1, value0) {
+                (Some(v1), Some(v0)) if v1 > v0 => acc + (v1 - v0),
+                _ => acc,
+            }
+        });
+
+        (elevation_gain_sum, String::from(units))
+    }
+
+    fn partition_stats_for_fields(
+        &self,
+        by_field: String,
+        partition_distance: f64,
+        fields: Vec<String>,
+    ) -> Vec<Vec<(String, (f64, String))>> {
+        // first get the partitions
+        let partition_indices = self.calculate_partition_indices(partition_distance, by_field);
+
+        partition_indices
+            .windows(2)
+            .map(|window| {
+                let start = window[0];
+                let end = window[1];
+                let partition_records = self.0[start..=end].to_vec();
+
+                // ok we have this, now we can map the resulf the field
+                let results: Vec<(String, (f64, String))> = fields
+                    .iter()
+                    .map(|field_name| {
+                        (
+                            field_name.clone(),
+                            self.avg_for_records(&partition_records, field_name.clone()),
+                        )
+                    })
+                    .collect();
+
+                results
+            })
+            .collect()
+    }
+
+    fn zone_time_for(&self, zones: Vec<(f64, f64)>, field_name: String) -> Vec<((f64, f64), f64)> {
+        // filter records
+        let filtered_records: Vec<&FitDataRecord> = self
+            .0
+            .iter()
+            .filter(|r| r.kind() == MesgNum::Record)
+            .filter(|r| r.fields().iter().any(|f| f.name() == field_name))
+            .collect();
+        self.zone_time_for_records(filtered_records, zones, field_name)
+    }
+
+    /// Given a list of records, and list of zones, calculate the time spent in each zone
+    fn zone_time_for_records(
+        &self,
+        records: Vec<&FitDataRecord>,
+        zones: Vec<(f64, f64)>,
+        field_name: String,
+    ) -> Vec<((f64, f64), f64)> {
+        let mut zone_times: Vec<((f64, f64), f64)> =
+            zones.iter().map(|z| (z.clone(), 0.0)).collect();
+
+        for window in records.windows(2) {
+            let value1 = window[1]
+                .fields()
+                .iter()
+                .find(|f| f.name() == field_name)
+                .unwrap()
+                .value();
+
+            // using MyValue to extract the value
+            let value1 = match MyValue(value1.clone()).as_f64() {
+                Some(v) => v,
+                None => continue,
+            };
+
+            // we need to find the zone from start again
+            // because the value could drop below the current zone
+            // always reset the current zone index to 0
+            let mut current_zone_index = 0;
+            while current_zone_index < zones.len() - 1 && value1 > zones[current_zone_index].1 {
+                // moving to next zone
+                // as current zone index is less than the last zone
+                // also the value is greater than the current zone's max
+                current_zone_index += 1;
+            }
+
+            // find the timestamp value
+            let timestamp1 = window[1]
+                .fields()
+                .iter()
+                .find(|f| f.name() == "timestamp")
+                .unwrap()
+                .value();
+
+            let timestamp1_value = match timestamp1 {
+                Value::Timestamp(t) => t.timestamp(),
+                _ => continue,
+            };
+
+            let timestamp0 = window[0]
+                .fields()
+                .iter()
+                .find(|f| f.name() == "timestamp")
+                .unwrap()
+                .value();
+
+            let timestamp0_value = match timestamp0 {
+                Value::Timestamp(t) => t.timestamp(),
+                _ => continue,
+            };
+
+            let time_diff = timestamp1_value - timestamp0_value;
+            zone_times[current_zone_index].1 += time_diff as f64;
+        }
+
+        zone_times
+    }
+
+    fn sample_series_for_records(
+        &self,
+        field_name: String,
+        num_of_points: u16,
+    ) -> Vec<(i64, f64, String)> {
+        self.0.sample_series_for_records(field_name, num_of_points)
+    }
+
+    /// Calculate the average for a given field name for a list of records
+    fn avg_for_records(&self, records: &Vec<FitDataRecord>, field_name: String) -> (f64, String) {
+        // only get the record types
+        let fields: Vec<&FitDataField> = records
+            .iter()
+            .filter(|r| r.kind() == MesgNum::Record)
+            .flat_map(|r| r.fields().iter().filter(|field| field.name() == field_name))
+            .collect();
+
+        // do a map filter to only sum the possible values could be sumed
+        // we only care about int, float values
+        let sumable_values: Vec<f64> = fields
+            .iter()
+            .filter_map(|field| MyValue(field.value().clone()).as_f64())
+            .collect();
+
+        let sum: f64 = sumable_values.iter().sum();
+        let count = sumable_values.len();
+
+        if count == 0 {
+            (0.0, String::from(""))
+        } else {
+            // we also need to return the unit
+            let units = fields.first().unwrap().units();
+            let avg_value = sum / count as f64;
+            (avg_value, String::from(units))
+        }
+    }
+
+    fn calculate_partition_indices(
+        &self,
+        partition_distance: f64,
+        field_name: String,
+    ) -> Vec<usize> {
+        let records: Vec<&FitDataRecord> = self
+            .0
+            .iter()
+            .filter(|r| r.kind() == MesgNum::Record)
+            .collect();
+        self.calculate_partition_indices_for_records(records, partition_distance, field_name)
+    }
+
+    /// Given a list of records, calculate the partition indices based on the field name and partition distance
+    fn calculate_partition_indices_for_records(
+        &self,
+        records: Vec<&FitDataRecord>,
+        partition_distance: f64,
+        field_name: String,
+    ) -> Vec<usize> {
+        let mut partition_indices = vec![0]; // always start include the start index
+        let mut start_distance = 0.0;
+
+        // let's loop
+        for (index, record) in records.iter().enumerate().skip(1) {
+            let fields: Vec<&FitDataField> = record
+                .fields()
+                .iter()
+                .filter(|f| f.name() == field_name)
+                .collect();
+
+            let distance_field = fields
+                .first()
+                .and_then(|f| MyValue(f.value().clone()).as_f64());
+            match distance_field {
+                Some(distance_value) => {
+                    if distance_value - start_distance >= partition_distance {
+                        // found it
+                        partition_indices.push(index);
+                        start_distance = distance_value;
+                    }
+                }
+                None => {}
+            }
+        }
+
+        // now we have the whole array
+        // if the last record is not there, add it
+        if *partition_indices.last().unwrap() != records.len() - 1 {
+            partition_indices.push(records.len() - 1);
+        }
+
+        partition_indices
+    }
+}
 
 // recursive method to turn Fit value into magnus::Value
-fn value_to_rb_value(value: &Value) -> magnus::Value {
+fn value_to_rb_value(value: &Value) -> Option<magnus::Value> {
     match value {
-        Value::Timestamp(t) => t.timestamp().into_value(),
-        Value::SInt8(i) => i.into_value(),
-        Value::UInt8(u) => u.into_value(),
-        Value::SInt16(i) => i.into_value(),
-        Value::UInt16(u) => u.into_value(),
-        Value::SInt32(i) => i.into_value(),
-        Value::UInt32(u) => u.into_value(),
-        Value::String(s) => s.clone().into_value(),
-        Value::Float32(f) => f.into_value(),
-        Value::Float64(f) => f.into_value(),
-        Value::UInt8z(u) => u.into_value(),
-        Value::UInt16z(u) => u.into_value(),
-        Value::UInt32z(u) => u.into_value(),
-        Value::Byte(b) => b.into_value(),
-        Value::SInt64(i) => i.into_value(),
-        Value::UInt64(u) => u.into_value(),
-        Value::UInt64z(u) => u.into_value(),
-        Value::Enum(e) => e.into_value(),
+        Value::Timestamp(t) => Some(t.timestamp().into_value()),
+        Value::SInt8(i) => Some(i.into_value()),
+        Value::UInt8(u) => Some(u.into_value()),
+        Value::SInt16(i) => Some(i.into_value()),
+        Value::UInt16(u) => Some(u.into_value()),
+        Value::SInt32(i) => Some(i.into_value()),
+        Value::UInt32(u) => Some(u.into_value()),
+        Value::String(s) => Some(s.clone().into_value()),
+        Value::Float32(f) => Some(f.into_value()),
+        Value::Float64(f) => Some(f.into_value()),
+        Value::UInt8z(u) => Some(u.into_value()),
+        Value::UInt16z(u) => Some(u.into_value()),
+        Value::UInt32z(u) => Some(u.into_value()),
+        Value::Byte(b) => Some(b.into_value()),
+        Value::SInt64(i) => Some(i.into_value()),
+        Value::UInt64(u) => Some(u.into_value()),
+        Value::UInt64z(u) => Some(u.into_value()),
+        Value::Enum(e) => Some(e.into_value()),
         Value::Array(arr) => {
             let rb_array = RArray::new();
             for value in arr {
                 rb_array.push(value_to_rb_value(value)).unwrap();
             }
-            rb_array.into_value()
+            Some(rb_array.into_value())
         }
+        Value::Invalid => None,
     }
 }
 
-///////////////////////// RFitDataRecord ///////////////////////////
-#[magnus::wrap(class = "RFitDataRecord")]
-struct RFitDataRecord(FitDataRecord);
-
-impl RFitDataRecord {
-    fn kind(&self) -> String {
-        self.0.kind().to_string()
-    }
-
-    fn fields_hash(&self) -> RHash {
-        let hash = RHash::new();
-        for field in self.0.fields() {
-            let value = value_to_rb_value(field.value());
-            let pair = RHash::new();
-            pair.aset(Symbol::new("units"), field.units()).unwrap();
-            pair.aset(Symbol::new("value"), value).unwrap();
-            // here we add the stuff to the hash
-            let field_name_symbol = Symbol::new(field.name());
-            hash.aset(field_name_symbol, pair).unwrap();
+// Turning FitDataRecord into a hash
+fn get_fields_hash(record: &FitDataRecord) -> RHash {
+    let hash = RHash::new();
+    for field in record.fields() {
+        match value_to_rb_value(field.value()) {
+            Some(value) => {
+                let pair = RHash::new();
+                pair.aset(Symbol::new("units"), field.units()).unwrap();
+                pair.aset(Symbol::new("value"), value).unwrap();
+                // here we add the stuff to the hash
+                let field_name_symbol = Symbol::new(field.name());
+                hash.aset(field_name_symbol, pair).unwrap();
+            }
+            None => {}
         }
-
-        hash
     }
+
+    hash
 }
 
 // Here we define two ruby classes
 // RFitDataRecord and RFitDataField
 fn define_ruby_classes(ruby: &Ruby) -> Result<(), magnus::Error> {
     // definie the the other one here
-    let data_record_class = ruby.define_class("RFitDataRecord", ruby.class_object())?;
-    data_record_class.define_method("kind", method!(RFitDataRecord::kind, 0))?;
-    data_record_class.define_method("fields_hash", method!(RFitDataRecord::fields_hash, 0))?;
+    let data_record_class = ruby.define_class("FitParseResult", ruby.class_object())?;
+    data_record_class.define_method("records_hash", method!(FitParseResult::records_hash, 0))?;
+    data_record_class.define_method("avg_for", method!(FitParseResult::avg_for, 1))?;
+    data_record_class
+        .define_method("elevation_gain", method!(FitParseResult::elevation_gain, 1))?;
+    data_record_class.define_method(
+        "calculate_partition_indices",
+        method!(FitParseResult::calculate_partition_indices, 2),
+    )?;
+
+    data_record_class.define_method(
+        "partition_stats_for_fields",
+        method!(FitParseResult::partition_stats_for_fields, 3),
+    )?;
+
+    data_record_class.define_method(
+        "sample_series_for_records",
+        method!(FitParseResult::sample_series_for_records, 2),
+    )?;
+
+    data_record_class.define_method("zone_time_for", method!(FitParseResult::zone_time_for, 2))?;
 
     Ok(())
 }
 
-fn parse_fit_file(file_path: String) -> Result<RArray, magnus::Error> {
+fn parse_fit_file(file_path: String) -> Result<FitParseResult, magnus::Error> {
     let mut fp = File::open(file_path)
         .map_err(|e| Error::new(Ruby::get().unwrap().exception_io_error(), e.to_string()))?;
     let data = fitparser::from_reader(&mut fp).map_err(|e| {
@@ -79,20 +528,16 @@ fn parse_fit_file(file_path: String) -> Result<RArray, magnus::Error> {
         )
     })?;
 
-    // finally we have the result array of record
-    let array = RArray::new();
-    for record in data {
-        array.push(RFitDataRecord(record)).unwrap();
-    }
+    let result = FitParseResult(data);
 
-    Ok(array)
+    Ok(result)
 }
 
 #[magnus::init]
 fn init(ruby: &Ruby) -> Result<(), Error> {
     let module = ruby.define_module("FitKit")?;
-    let _ = define_ruby_classes(&ruby);
 
+    let _ = define_ruby_classes(ruby);
     module.define_singleton_method("parse_fit_file", function!(parse_fit_file, 1))?;
 
     Ok(())

data/lib/fit_kit/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module FitKit
-  VERSION = "0.2.0"
+  VERSION = "0.4.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fit_kit
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.4.0
 platform: ruby
 authors:
 - 29decibel
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-10-16 00:00:00.000000000 Z
+date: 2025-03-17 00:00:00.000000000 Z
 dependencies: []
 description: Garmin fit file parser wrapping Rust crate fitparse_rs.
 email: