@sports-alliance/sports-lib 6.1.9 → 6.1.11

package/babel.config.js

@@ -1,3 +1,3 @@
 module.exports = {
-    presets: [['@babel/preset-env', { targets: { node: 'current' } }]],
+  presets: [['@babel/preset-env', { targets: { node: 'current' } }]]
 };

package/lib/cjs/activities/activity-parsing-options.d.ts

@@ -15,5 +15,5 @@ export declare class ActivityParsingOptions {
     };
     maxActivityDurationDays: number;
     generateUnitStreams: boolean;
-    constructor(options: ActivityParsingOptions);
+    constructor(options: Partial<ActivityParsingOptions>);
 }

package/lib/cjs/activities/activity-parsing-options.js

@@ -3,10 +3,13 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.ActivityParsingOptions = void 0;
 class ActivityParsingOptions {
     constructor(options) {
-        var _a;
-        this.streams = options.streams;
-        this.maxActivityDurationDays = options.maxActivityDurationDays;
-        this.generateUnitStreams = (_a = options.generateUnitStreams) !== null && _a !== void 0 ? _a : true;
+        var _a, _b, _c;
+        this.streams = (_a = options.streams) !== null && _a !== void 0 ? _a : {
+            smooth: { altitudeSmooth: true, grade: true, gradeSmooth: true },
+            fixAbnormal: { speed: false }
+        };
+        this.maxActivityDurationDays = (_b = options.maxActivityDurationDays) !== null && _b !== void 0 ? _b : 14;
+        this.generateUnitStreams = (_c = options.generateUnitStreams) !== null && _c !== void 0 ? _c : true;
     }
 }
 exports.ActivityParsingOptions = ActivityParsingOptions;

package/lib/cjs/data/data-store.spec.js

@@ -70,7 +70,9 @@ describe('DataStore', () => {
                 temperatureUnits: [],
                 weightUnits: []
             };
-            const result = data_store_1.DynamicDataLoader.getUnitBasedDataTypesFromDataTypes([data_speed_1.DataSpeed.type], settings, { includeDerivedTypes: false });
+            const result = data_store_1.DynamicDataLoader.getUnitBasedDataTypesFromDataTypes([data_speed_1.DataSpeed.type], settings, {
+                includeDerivedTypes: false
+            });
             expect(result).toContain(data_speed_1.DataSpeedKilometersPerHour.type);
             expect(result).not.toContain(data_pace_1.DataPaceMinutesPerMile.type);
         });

package/lib/cjs/events/adapters/importers/fit/importer.fit.js

@@ -627,10 +627,20 @@ class EventImporterFIT {
         // Pause TIME on Object (activity, lap...)
         const pause = elapsedTime > movingTime && movingTime > 0 ? Math.round((elapsedTime - movingTime) * 100) / 100 : 0;
         stats.push(new data_pause_1.DataPause(pause));
+        const getStatValue = (obj, keys) => {
+            for (const key of keys) {
+                if ((0, helpers_1.isNumberOrString)(obj[key])) {
+                    return obj[key];
+                }
+            }
+            return null;
+        };
+        const avgSpeed = getStatValue(object, ['enhanced_avg_speed', 'EnhancedAvgSpeed', 'avg_speed', 'AvgSpeed']);
+        const totalDistance = getStatValue(object, ['total_distance', 'TotalDistance']);
         // Assign is active lap status
-        stats.push(new data_active_lap_1.DataActiveLap(!!(object.total_distance || object.avg_speed)));
-        if ((0, helpers_1.isNumberOrString)(object.total_distance)) {
-            stats.push(new data_distance_1.DataDistance(object.total_distance));
+        stats.push(new data_active_lap_1.DataActiveLap(!!(totalDistance || avgSpeed)));
+        if (totalDistance !== null) {
+            stats.push(new data_distance_1.DataDistance(totalDistance));
         }
         else {
             stats.push(new data_distance_1.DataDistance(0));
@@ -690,24 +700,16 @@ class EventImporterFIT {
             stats.push(new data_power_pedal_smoothness_right_1.DataPowerPedalSmoothnessRight(object.avg_right_pedal_smoothness));
         }
         // Speed
-        if ((0, helpers_1.isNumberOrString)(object.avg_speed)) {
-            stats.push(new data_speed_avg_1.DataSpeedAvg(object.avg_speed));
-        }
-        if ((0, helpers_1.isNumberOrString)(object.min_speed)) {
-            stats.push(new data_speed_min_1.DataSpeedMin(object.min_speed));
-        }
-        if ((0, helpers_1.isNumberOrString)(object.max_speed)) {
-            stats.push(new data_speed_max_1.DataSpeedMax(object.max_speed));
-        }
-        // Keep latest , enhanced @todo this can create a bug
-        if ((0, helpers_1.isNumberOrString)(object.enhanced_avg_speed)) {
-            stats.push(new data_speed_avg_1.DataSpeedAvg(object.enhanced_avg_speed));
+        if (avgSpeed !== null) {
+            stats.push(new data_speed_avg_1.DataSpeedAvg(avgSpeed));
         }
-        if ((0, helpers_1.isNumberOrString)(object.enhanced_min_speed)) {
-            stats.push(new data_speed_min_1.DataSpeedMin(object.enhanced_min_speed));
+        const minSpeed = getStatValue(object, ['enhanced_min_speed', 'EnhancedMinSpeed', 'min_speed', 'MinSpeed']);
+        if (minSpeed !== null) {
+            stats.push(new data_speed_min_1.DataSpeedMin(minSpeed));
         }
-        if ((0, helpers_1.isNumberOrString)(object.enhanced_max_speed)) {
-            stats.push(new data_speed_max_1.DataSpeedMax(object.enhanced_max_speed));
+        const maxSpeed = getStatValue(object, ['enhanced_max_speed', 'EnhancedMaxSpeed', 'max_speed', 'MaxSpeed']);
+        if (maxSpeed !== null) {
+            stats.push(new data_speed_max_1.DataSpeedMax(maxSpeed));
         }
         // Temperature
         if ((0, helpers_1.isNumberOrString)(object.avg_temperature)) {
@@ -720,12 +722,14 @@ class EventImporterFIT {
             stats.push(new data_temperature_max_1.DataTemperatureMax(object.max_temperature));
         }
         // Ascent
-        if ((0, helpers_1.isNumberOrString)(object.total_ascent)) {
-            stats.push(new data_ascent_1.DataAscent(object.total_ascent));
+        const ascent = getStatValue(object, ['total_ascent', 'TotalAscent']);
+        if (ascent !== null) {
+            stats.push(new data_ascent_1.DataAscent(ascent));
         }
         // Descent
-        if ((0, helpers_1.isNumberOrString)(object.total_descent)) {
-            stats.push(new data_descent_1.DataDescent(object.total_descent));
+        const descent = getStatValue(object, ['total_descent', 'TotalDescent']);
+        if (descent !== null) {
+            stats.push(new data_descent_1.DataDescent(descent));
         }
         // Calories
         if ((0, helpers_1.isNumberOrString)(object.total_calories)) {

package/lib/cjs/events/adapters/importers/fit/importer.fit.mapper.js

@@ -50,7 +50,7 @@ exports.FITSampleMapper = [
     {
         dataType: data_distance_1.DataDistance.type,
         getSampleValue: (sample) => {
-            return sample.distance;
+            return (0, helpers_1.isNumber)(sample.distance) ? sample.distance : sample.Distance;
         }
     },
     {
@@ -62,13 +62,19 @@ exports.FITSampleMapper = [
     {
         dataType: data_altitude_1.DataAltitude.type,
         getSampleValue: (sample) => {
-            return (0, helpers_1.isNumber)(sample.enhanced_altitude)
-                ? Math.round(sample.enhanced_altitude * Math.pow(10, constants_1.ALTITUDE_PRECISION_NUMBER_OF_DECIMAL_PLACES)) /
+            const altitude = (0, helpers_1.isNumber)(sample.enhanced_altitude)
+                ? sample.enhanced_altitude
+                : (0, helpers_1.isNumber)(sample.EnhancedAltitude)
+                    ? sample.EnhancedAltitude
+                    : (0, helpers_1.isNumber)(sample.altitude)
+                        ? sample.altitude
+                        : (0, helpers_1.isNumber)(sample.Altitude)
+                            ? sample.Altitude
+                            : null;
+            return (0, helpers_1.isNumber)(altitude)
+                ? Math.round(altitude * Math.pow(10, constants_1.ALTITUDE_PRECISION_NUMBER_OF_DECIMAL_PLACES)) /
                     Math.pow(10, constants_1.ALTITUDE_PRECISION_NUMBER_OF_DECIMAL_PLACES)
-                : (0, helpers_1.isNumber)(sample.altitude)
-                    ? Math.round(sample.altitude * Math.pow(10, constants_1.ALTITUDE_PRECISION_NUMBER_OF_DECIMAL_PLACES)) /
-                        Math.pow(10, constants_1.ALTITUDE_PRECISION_NUMBER_OF_DECIMAL_PLACES)
-                    : sample.altitude;
+                : altitude;
         }
     },
     {
@@ -105,12 +111,18 @@ exports.FITSampleMapper = [
     {
         dataType: data_speed_1.DataSpeed.type,
         getSampleValue: (sample) => {
-            if (Number.isFinite(sample.enhanced_speed)) {
+            if ((0, helpers_1.isNumber)(sample.enhanced_speed)) {
                 return sample.enhanced_speed;
             }
-            if (Number.isFinite(sample.speed)) {
+            if ((0, helpers_1.isNumber)(sample.EnhancedSpeed)) {
+                return sample.EnhancedSpeed;
+            }
+            if ((0, helpers_1.isNumber)(sample.speed)) {
                 return sample.speed;
             }
+            if ((0, helpers_1.isNumber)(sample.Speed)) {
+                return sample.Speed;
+            }
             return null;
         }
     },

package/lib/cjs/events/utilities/activity.utilities.js

@@ -291,9 +291,12 @@ class ActivityUtilities {
     static generateMissingStreams(activity) {
         // Compute missing streams
         this.generateMissingStreamsForActivity(activity);
-        if (!activity.parseOptions || activity.parseOptions.generateUnitStreams) {
-            activity.addStreams(this.createUnitStreamsFromStreams(activity.getAllStreams(), activity.type));
-        }
+        // Always include derived base streams (like Pace), but conditionally include unit variants
+        const includeUnitVariants = !activity.parseOptions || activity.parseOptions.generateUnitStreams;
+        activity.addStreams(this.createUnitStreamsFromStreams(activity.getAllStreams(), activity.type, undefined, {
+            includeDerivedTypes: true,
+            includeUnitVariants
+        }));
     }
     static getSummaryStatsForActivities(activities) {
         const stats = [];
@@ -680,7 +683,10 @@ class ActivityUtilities {
         includeUnitVariants: true
     }) {
         // @todo perhaps check input to be unitStreamTypesStrictly
-        const unitStreamTypesToCreate = unitStreamTypes || data_store_1.DynamicDataLoader.allUnitDerivedDataTypes;
+        const unitStreamTypesToCreate = unitStreamTypes || [
+            ...data_store_1.DynamicDataLoader.allUnitDerivedDataTypes,
+            ...data_store_1.DynamicDataLoader.speedDerivedDataTypes
+        ];
         let baseUnitStreams = [];
         const speedStream = streams.find(stream => stream.type === data_speed_1.DataSpeed.type);
         if (speedStream) {
@@ -707,8 +713,7 @@ class ActivityUtilities {
                 ? baseUnitStreams.concat(verticalSpeedStream)
                 : baseUnitStreams;
         }
-        // @todo add distance ?
-        const startWith = baseUnitStreams.filter(baseUnitStream => unitStreamTypesToCreate.indexOf(baseUnitStream.type) !== -1 && streams.indexOf(baseUnitStream) === -1);
+        const startWith = baseUnitStreams.filter((baseUnitStream) => unitStreamTypesToCreate.indexOf(baseUnitStream.type) !== -1 && streams.indexOf(baseUnitStream) === -1);
         if (options.includeUnitVariants === false) {
             return startWith;
         }
@@ -867,7 +872,8 @@ class ActivityUtilities {
             activity.addStream(leftPowerStream);
         }
         // If left stance time stream available, then add the right balance stream too
-        if (activity.hasStreamData(data_stance_time_balance_left_1.DataStanceTimeBalanceLeft.type) && !activity.hasStreamData(data_stance_time_balance_right_1.DataStanceTimeBalanceRight.type)) {
+        if (activity.hasStreamData(data_stance_time_balance_left_1.DataStanceTimeBalanceLeft.type) &&
+            !activity.hasStreamData(data_stance_time_balance_right_1.DataStanceTimeBalanceRight.type)) {
             const rightStanceBalanceTimeStream = activity.createStream(data_stance_time_balance_right_1.DataStanceTimeBalanceRight.type);
             const leftStanceBalanceTimeStream = activity.getStreamData(data_stance_time_balance_left_1.DataStanceTimeBalanceLeft.type);
             const rightStanceBalanceTimeData = leftStanceBalanceTimeStream.map(leftBalance => {

package/lib/cjs/events/utilities/activity.utilities.spec.js

@@ -238,7 +238,10 @@ describe('Activity Utilities', () => {
         });
         it('should exclude derived types when includeDerivedTypes is false', () => {
             const streams = [new stream_1.Stream(data_speed_1.DataSpeed.type, [10, 20])];
-            const result = activity_utilities_1.ActivityUtilities.createUnitStreamsFromStreams(streams, activity_types_1.ActivityTypes.Running, undefined, { includeDerivedTypes: false, includeUnitVariants: true });
+            const result = activity_utilities_1.ActivityUtilities.createUnitStreamsFromStreams(streams, activity_types_1.ActivityTypes.Running, undefined, {
+                includeDerivedTypes: false,
+                includeUnitVariants: true
+            });
             const paceStream = result.find(s => s.type === data_pace_1.DataPaceMinutesPerMile.type);
             const kmhStream = result.find(s => s.type === data_speed_2.DataSpeedKilometersPerHour.type);
             expect(paceStream).toBeUndefined();
@@ -247,7 +250,10 @@ describe('Activity Utilities', () => {
         it('should exclude unit variants when includeUnitVariants is false', () => {
             const streams = [new stream_1.Stream(data_speed_1.DataSpeed.type, [10, 20])];
             // We pass DataPace.type in unitStreamTypes so we can check if the derived base stream is present
-            const result = activity_utilities_1.ActivityUtilities.createUnitStreamsFromStreams(streams, activity_types_1.ActivityTypes.Running, [data_pace_1.DataPace.type], { includeDerivedTypes: true, includeUnitVariants: false });
+            const result = activity_utilities_1.ActivityUtilities.createUnitStreamsFromStreams(streams, activity_types_1.ActivityTypes.Running, [data_pace_1.DataPace.type], {
+                includeDerivedTypes: true,
+                includeUnitVariants: false
+            });
             const paceStream = result.find(s => s.type === data_pace_1.DataPace.type);
             const paceUnitStream = result.find(s => s.type === data_pace_1.DataPaceMinutesPerMile.type);
             expect(paceStream).toBeDefined(); // Derived type should be there
@@ -255,7 +261,10 @@ describe('Activity Utilities', () => {
         });
         it('should exclude both derived types and unit variants', () => {
             const streams = [new stream_1.Stream(data_speed_1.DataSpeed.type, [10, 20])];
-            const result = activity_utilities_1.ActivityUtilities.createUnitStreamsFromStreams(streams, activity_types_1.ActivityTypes.Running, [data_pace_1.DataPace.type], { includeDerivedTypes: false, includeUnitVariants: false });
+            const result = activity_utilities_1.ActivityUtilities.createUnitStreamsFromStreams(streams, activity_types_1.ActivityTypes.Running, [data_pace_1.DataPace.type], {
+                includeDerivedTypes: false,
+                includeUnitVariants: false
+            });
             const paceStream = result.find(s => s.type === data_pace_1.DataPace.type);
             const paceUnitStream = result.find(s => s.type === data_pace_1.DataPaceMinutesPerMile.type);
             expect(paceStream).toBeUndefined();
@@ -278,7 +287,7 @@ describe('Activity Utilities', () => {
             activity.parseOptions = {
                 streams: { smooth: {}, fixAbnormal: {} },
                 maxActivityDurationDays: 14,
-                generateUnitStreams: false,
+                generateUnitStreams: false
             };
             // Add a speed stream
             activity.addStream(new stream_1.Stream(data_speed_1.DataSpeed.type, [10, 20]));
@@ -296,20 +305,18 @@ describe('Activity Utilities', () => {
             activity.parseOptions = {
                 streams: { smooth: {}, fixAbnormal: {} },
                 maxActivityDurationDays: 14,
-                generateUnitStreams: false, // DISABLE streams
+                generateUnitStreams: false // DISABLE streams
             };
             // Add a speed stream [10 m/s, 20 m/s]
             // Max speed = 20 m/s
             activity.addStream(new stream_1.Stream(data_speed_1.DataSpeed.type, [10, 20]));
             activity_utilities_1.ActivityUtilities.generateMissingStreamsAndStatsForActivity(activity);
-            // 1. Verify Streams are missing (as requested)
-            expect(activity.hasStreamData(data_speed_2.DataSpeedKilometersPerHour.type)).toBe(false);
-            // 1. Verify Streams are missing (as requested)
+            // 1. Verify Unit Streams are missing (as requested)
             expect(activity.hasStreamData(data_speed_2.DataSpeedKilometersPerHour.type)).toBe(false);
-            // 2. Verify Stats are PRESENT (Safety Check)
+            // 2. Verify Derived Base Streams are PRESENT (The fix)
+            expect(activity.hasStreamData(data_pace_1.DataPace.type)).toBe(true);
+            // 3. Verify Stats are PRESENT (Safety Check)
             // 20 m/s = 72 km/h
-            // We check for the stat by its string type if we can't import the constant easily, or iterate.
-            // Based on activity.utilities.ts it uses: DataSpeedMaxKilometersPerHour
             const allStats = Array.from(activity.getStats().values());
             const speedMaxKmh = allStats.find(s => s.getType() === data_speed_max_1.DataSpeedMaxKilometersPerHour.type);
             expect(speedMaxKmh).toBeDefined();

package/lib/esm/activities/activity-parsing-options.d.ts

@@ -15,5 +15,5 @@ export declare class ActivityParsingOptions {
     };
     maxActivityDurationDays: number;
     generateUnitStreams: boolean;
-    constructor(options: ActivityParsingOptions);
+    constructor(options: Partial<ActivityParsingOptions>);
 }

package/lib/esm/activities/activity-parsing-options.js

@@ -1,9 +1,12 @@
 export class ActivityParsingOptions {
     constructor(options) {
-        var _a;
-        this.streams = options.streams;
-        this.maxActivityDurationDays = options.maxActivityDurationDays;
-        this.generateUnitStreams = (_a = options.generateUnitStreams) !== null && _a !== void 0 ? _a : true;
+        var _a, _b, _c;
+        this.streams = (_a = options.streams) !== null && _a !== void 0 ? _a : {
+            smooth: { altitudeSmooth: true, grade: true, gradeSmooth: true },
+            fixAbnormal: { speed: false }
+        };
+        this.maxActivityDurationDays = (_b = options.maxActivityDurationDays) !== null && _b !== void 0 ? _b : 14;
+        this.generateUnitStreams = (_c = options.generateUnitStreams) !== null && _c !== void 0 ? _c : true;
     }
 }
 ActivityParsingOptions.DEFAULT = new ActivityParsingOptions({

package/lib/esm/data/data-store.spec.js

@@ -68,7 +68,9 @@ describe('DataStore', () => {
                 temperatureUnits: [],
                 weightUnits: []
             };
-            const result = DynamicDataLoader.getUnitBasedDataTypesFromDataTypes([DataSpeed.type], settings, { includeDerivedTypes: false });
+            const result = DynamicDataLoader.getUnitBasedDataTypesFromDataTypes([DataSpeed.type], settings, {
+                includeDerivedTypes: false
+            });
             expect(result).toContain(DataSpeedKilometersPerHour.type);
             expect(result).not.toContain(DataPaceMinutesPerMile.type);
         });

package/lib/esm/events/adapters/importers/fit/importer.fit.js

@@ -621,10 +621,20 @@ export class EventImporterFIT {
         // Pause TIME on Object (activity, lap...)
         const pause = elapsedTime > movingTime && movingTime > 0 ? Math.round((elapsedTime - movingTime) * 100) / 100 : 0;
         stats.push(new DataPause(pause));
+        const getStatValue = (obj, keys) => {
+            for (const key of keys) {
+                if (isNumberOrString(obj[key])) {
+                    return obj[key];
+                }
+            }
+            return null;
+        };
+        const avgSpeed = getStatValue(object, ['enhanced_avg_speed', 'EnhancedAvgSpeed', 'avg_speed', 'AvgSpeed']);
+        const totalDistance = getStatValue(object, ['total_distance', 'TotalDistance']);
         // Assign is active lap status
-        stats.push(new DataActiveLap(!!(object.total_distance || object.avg_speed)));
-        if (isNumberOrString(object.total_distance)) {
-            stats.push(new DataDistance(object.total_distance));
+        stats.push(new DataActiveLap(!!(totalDistance || avgSpeed)));
+        if (totalDistance !== null) {
+            stats.push(new DataDistance(totalDistance));
         }
         else {
             stats.push(new DataDistance(0));
@@ -684,24 +694,16 @@ export class EventImporterFIT {
             stats.push(new DataPowerPedalSmoothnessRight(object.avg_right_pedal_smoothness));
         }
         // Speed
-        if (isNumberOrString(object.avg_speed)) {
-            stats.push(new DataSpeedAvg(object.avg_speed));
-        }
-        if (isNumberOrString(object.min_speed)) {
-            stats.push(new DataSpeedMin(object.min_speed));
-        }
-        if (isNumberOrString(object.max_speed)) {
-            stats.push(new DataSpeedMax(object.max_speed));
-        }
-        // Keep latest , enhanced @todo this can create a bug
-        if (isNumberOrString(object.enhanced_avg_speed)) {
-            stats.push(new DataSpeedAvg(object.enhanced_avg_speed));
+        if (avgSpeed !== null) {
+            stats.push(new DataSpeedAvg(avgSpeed));
         }
-        if (isNumberOrString(object.enhanced_min_speed)) {
-            stats.push(new DataSpeedMin(object.enhanced_min_speed));
+        const minSpeed = getStatValue(object, ['enhanced_min_speed', 'EnhancedMinSpeed', 'min_speed', 'MinSpeed']);
+        if (minSpeed !== null) {
+            stats.push(new DataSpeedMin(minSpeed));
         }
-        if (isNumberOrString(object.enhanced_max_speed)) {
-            stats.push(new DataSpeedMax(object.enhanced_max_speed));
+        const maxSpeed = getStatValue(object, ['enhanced_max_speed', 'EnhancedMaxSpeed', 'max_speed', 'MaxSpeed']);
+        if (maxSpeed !== null) {
+            stats.push(new DataSpeedMax(maxSpeed));
         }
         // Temperature
         if (isNumberOrString(object.avg_temperature)) {
@@ -714,12 +716,14 @@ export class EventImporterFIT {
             stats.push(new DataTemperatureMax(object.max_temperature));
         }
         // Ascent
-        if (isNumberOrString(object.total_ascent)) {
-            stats.push(new DataAscent(object.total_ascent));
+        const ascent = getStatValue(object, ['total_ascent', 'TotalAscent']);
+        if (ascent !== null) {
+            stats.push(new DataAscent(ascent));
         }
         // Descent
-        if (isNumberOrString(object.total_descent)) {
-            stats.push(new DataDescent(object.total_descent));
+        const descent = getStatValue(object, ['total_descent', 'TotalDescent']);
+        if (descent !== null) {
+            stats.push(new DataDescent(descent));
         }
         // Calories
         if (isNumberOrString(object.total_calories)) {

package/lib/esm/events/adapters/importers/fit/importer.fit.mapper.js

@@ -47,7 +47,7 @@ export const FITSampleMapper = [
     {
         dataType: DataDistance.type,
         getSampleValue: (sample) => {
-            return sample.distance;
+            return isNumber(sample.distance) ? sample.distance : sample.Distance;
         }
     },
     {
@@ -59,13 +59,19 @@ export const FITSampleMapper = [
     {
         dataType: DataAltitude.type,
         getSampleValue: (sample) => {
-            return isNumber(sample.enhanced_altitude)
-                ? Math.round(sample.enhanced_altitude * Math.pow(10, ALTITUDE_PRECISION_NUMBER_OF_DECIMAL_PLACES)) /
+            const altitude = isNumber(sample.enhanced_altitude)
+                ? sample.enhanced_altitude
+                : isNumber(sample.EnhancedAltitude)
+                    ? sample.EnhancedAltitude
+                    : isNumber(sample.altitude)
+                        ? sample.altitude
+                        : isNumber(sample.Altitude)
+                            ? sample.Altitude
+                            : null;
+            return isNumber(altitude)
+                ? Math.round(altitude * Math.pow(10, ALTITUDE_PRECISION_NUMBER_OF_DECIMAL_PLACES)) /
                     Math.pow(10, ALTITUDE_PRECISION_NUMBER_OF_DECIMAL_PLACES)
-                : isNumber(sample.altitude)
-                    ? Math.round(sample.altitude * Math.pow(10, ALTITUDE_PRECISION_NUMBER_OF_DECIMAL_PLACES)) /
-                        Math.pow(10, ALTITUDE_PRECISION_NUMBER_OF_DECIMAL_PLACES)
-                    : sample.altitude;
+                : altitude;
         }
     },
     {
@@ -102,12 +108,18 @@ export const FITSampleMapper = [
     {
         dataType: DataSpeed.type,
         getSampleValue: (sample) => {
-            if (Number.isFinite(sample.enhanced_speed)) {
+            if (isNumber(sample.enhanced_speed)) {
                 return sample.enhanced_speed;
             }
-            if (Number.isFinite(sample.speed)) {
+            if (isNumber(sample.EnhancedSpeed)) {
+                return sample.EnhancedSpeed;
+            }
+            if (isNumber(sample.speed)) {
                 return sample.speed;
             }
+            if (isNumber(sample.Speed)) {
+                return sample.Speed;
+            }
             return null;
         }
     },

package/lib/esm/events/utilities/activity.utilities.js

@@ -288,9 +288,12 @@ export class ActivityUtilities {
     static generateMissingStreams(activity) {
         // Compute missing streams
         this.generateMissingStreamsForActivity(activity);
-        if (!activity.parseOptions || activity.parseOptions.generateUnitStreams) {
-            activity.addStreams(this.createUnitStreamsFromStreams(activity.getAllStreams(), activity.type));
-        }
+        // Always include derived base streams (like Pace), but conditionally include unit variants
+        const includeUnitVariants = !activity.parseOptions || activity.parseOptions.generateUnitStreams;
+        activity.addStreams(this.createUnitStreamsFromStreams(activity.getAllStreams(), activity.type, undefined, {
+            includeDerivedTypes: true,
+            includeUnitVariants
+        }));
     }
     static getSummaryStatsForActivities(activities) {
         const stats = [];
@@ -677,7 +680,10 @@ export class ActivityUtilities {
         includeUnitVariants: true
     }) {
         // @todo perhaps check input to be unitStreamTypesStrictly
-        const unitStreamTypesToCreate = unitStreamTypes || DynamicDataLoader.allUnitDerivedDataTypes;
+        const unitStreamTypesToCreate = unitStreamTypes || [
+            ...DynamicDataLoader.allUnitDerivedDataTypes,
+            ...DynamicDataLoader.speedDerivedDataTypes
+        ];
         let baseUnitStreams = [];
         const speedStream = streams.find(stream => stream.type === DataSpeed.type);
         if (speedStream) {
@@ -704,8 +710,7 @@ export class ActivityUtilities {
                 ? baseUnitStreams.concat(verticalSpeedStream)
                 : baseUnitStreams;
         }
-        // @todo add distance ?
-        const startWith = baseUnitStreams.filter(baseUnitStream => unitStreamTypesToCreate.indexOf(baseUnitStream.type) !== -1 && streams.indexOf(baseUnitStream) === -1);
+        const startWith = baseUnitStreams.filter((baseUnitStream) => unitStreamTypesToCreate.indexOf(baseUnitStream.type) !== -1 && streams.indexOf(baseUnitStream) === -1);
         if (options.includeUnitVariants === false) {
             return startWith;
         }
@@ -864,7 +869,8 @@ export class ActivityUtilities {
             activity.addStream(leftPowerStream);
         }
         // If left stance time stream available, then add the right balance stream too
-        if (activity.hasStreamData(DataStanceTimeBalanceLeft.type) && !activity.hasStreamData(DataStanceTimeBalanceRight.type)) {
+        if (activity.hasStreamData(DataStanceTimeBalanceLeft.type) &&
+            !activity.hasStreamData(DataStanceTimeBalanceRight.type)) {
             const rightStanceBalanceTimeStream = activity.createStream(DataStanceTimeBalanceRight.type);
             const leftStanceBalanceTimeStream = activity.getStreamData(DataStanceTimeBalanceLeft.type);
             const rightStanceBalanceTimeData = leftStanceBalanceTimeStream.map(leftBalance => {

package/lib/esm/events/utilities/activity.utilities.spec.js

@@ -236,7 +236,10 @@ describe('Activity Utilities', () => {
         });
         it('should exclude derived types when includeDerivedTypes is false', () => {
             const streams = [new Stream(DataSpeed.type, [10, 20])];
-            const result = ActivityUtilities.createUnitStreamsFromStreams(streams, ActivityTypes.Running, undefined, { includeDerivedTypes: false, includeUnitVariants: true });
+            const result = ActivityUtilities.createUnitStreamsFromStreams(streams, ActivityTypes.Running, undefined, {
+                includeDerivedTypes: false,
+                includeUnitVariants: true
+            });
             const paceStream = result.find(s => s.type === DataPaceMinutesPerMile.type);
             const kmhStream = result.find(s => s.type === DataSpeedKilometersPerHour.type);
             expect(paceStream).toBeUndefined();
@@ -245,7 +248,10 @@ describe('Activity Utilities', () => {
         it('should exclude unit variants when includeUnitVariants is false', () => {
             const streams = [new Stream(DataSpeed.type, [10, 20])];
             // We pass DataPace.type in unitStreamTypes so we can check if the derived base stream is present
-            const result = ActivityUtilities.createUnitStreamsFromStreams(streams, ActivityTypes.Running, [DataPace.type], { includeDerivedTypes: true, includeUnitVariants: false });
+            const result = ActivityUtilities.createUnitStreamsFromStreams(streams, ActivityTypes.Running, [DataPace.type], {
+                includeDerivedTypes: true,
+                includeUnitVariants: false
+            });
             const paceStream = result.find(s => s.type === DataPace.type);
             const paceUnitStream = result.find(s => s.type === DataPaceMinutesPerMile.type);
             expect(paceStream).toBeDefined(); // Derived type should be there
@@ -253,7 +259,10 @@ describe('Activity Utilities', () => {
         });
         it('should exclude both derived types and unit variants', () => {
             const streams = [new Stream(DataSpeed.type, [10, 20])];
-            const result = ActivityUtilities.createUnitStreamsFromStreams(streams, ActivityTypes.Running, [DataPace.type], { includeDerivedTypes: false, includeUnitVariants: false });
+            const result = ActivityUtilities.createUnitStreamsFromStreams(streams, ActivityTypes.Running, [DataPace.type], {
+                includeDerivedTypes: false,
+                includeUnitVariants: false
+            });
             const paceStream = result.find(s => s.type === DataPace.type);
             const paceUnitStream = result.find(s => s.type === DataPaceMinutesPerMile.type);
             expect(paceStream).toBeUndefined();
@@ -276,7 +285,7 @@ describe('Activity Utilities', () => {
             activity.parseOptions = {
                 streams: { smooth: {}, fixAbnormal: {} },
                 maxActivityDurationDays: 14,
-                generateUnitStreams: false,
+                generateUnitStreams: false
             };
             // Add a speed stream
             activity.addStream(new Stream(DataSpeed.type, [10, 20]));
@@ -294,20 +303,18 @@ describe('Activity Utilities', () => {
             activity.parseOptions = {
                 streams: { smooth: {}, fixAbnormal: {} },
                 maxActivityDurationDays: 14,
-                generateUnitStreams: false, // DISABLE streams
+                generateUnitStreams: false // DISABLE streams
             };
             // Add a speed stream [10 m/s, 20 m/s]
             // Max speed = 20 m/s
             activity.addStream(new Stream(DataSpeed.type, [10, 20]));
             ActivityUtilities.generateMissingStreamsAndStatsForActivity(activity);
-            // 1. Verify Streams are missing (as requested)
-            expect(activity.hasStreamData(DataSpeedKilometersPerHour.type)).toBe(false);
-            // 1. Verify Streams are missing (as requested)
+            // 1. Verify Unit Streams are missing (as requested)
             expect(activity.hasStreamData(DataSpeedKilometersPerHour.type)).toBe(false);
-            // 2. Verify Stats are PRESENT (Safety Check)
+            // 2. Verify Derived Base Streams are PRESENT (The fix)
+            expect(activity.hasStreamData(DataPace.type)).toBe(true);
+            // 3. Verify Stats are PRESENT (Safety Check)
             // 20 m/s = 72 km/h
-            // We check for the stat by its string type if we can't import the constant easily, or iterate.
-            // Based on activity.utilities.ts it uses: DataSpeedMaxKilometersPerHour
             const allStats = Array.from(activity.getStats().values());
             const speedMaxKmh = allStats.find(s => s.getType() === DataSpeedMaxKilometersPerHour.type);
             expect(speedMaxKmh).toBeDefined();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sports-alliance/sports-lib",
-  "version": "6.1.9",
+  "version": "6.1.11",
   "description": "A Library to for importing / exporting and processing GPX, TCX, FIT and JSON files from services such as Strava, Movescount, Garmin, Polar etc",
   "keywords": [
     "gpx",