search

sumoDave / GarminDiveFitAppender

Combines sequential dives into a single dive activity.
GNU General Public License v3.0
4 stars 0 forks source link

Error trying to merge 3 .fit files - #4

Closed chris-obermeier closed 4 months ago

chris-obermeier commented 4 months ago

I'm trying to merge 3 files but the GarminDiveFITTool throws the following errors.

Attached are the zipped 3 fit files I'm trying to merge: Fit Files.zip

Processing file: \Fit Files\15949026647_ACTIVITY.fit
Processing file: \Fit Files\15949026648_ACTIVITY.fit
Processing file: \Fit Files\15949026650_ACTIVITY.fit
Exception in thread "main" java.lang.NullPointerException: Cannot invoke "java.lang.Long.longValue()" because the return value of "com.garmin.fit.DiveSummaryMesg.getDiveNumber()" is null
Processing file: C:\Users\ChrisObermeier\Desktop\Fit Files\15949026647_ACTIVITY.fit
Processing file: C:\Users\ChrisObermeier\Desktop\Fit Files\15949026648_ACTIVITY.fit
Processing file: C:\Users\ChrisObermeier\Desktop\Fit Files\15949026650_ACTIVITY.fit
Exception in thread "main" java.lang.NullPointerException: Cannot invoke "java.lang.Long.longValue()" because the return value of "com.garmin.fit.DiveSummaryMesg.getDiveNumber()" is null
        at DiveProfile.setDiveNumberCnsN2O2Values(DiveProfile.java:88)
        at DiveProfile.initialiseData(DiveProfile.java:22)
        at Main.main(Main.java:29)
sumoDave commented 4 months ago

Thanks for attaching the files . Seems that Order of Dive Summary messages have changed in fit files generated by the MK3. Will have a fix in soon.

chris-obermeier commented 4 months ago

Thank you!

Thanks,

Chris Obermeier

Head of Engineering

[Logo Description automatically generated]

—

Remote

Email: @.**@.>

Website: xpanse.com

Phone: 414.429.4185

[A picture containing company name Description automatically generated] [A picture containing icon Description automatically generated]

From: sumoDave @.> Sent: Sunday, June 16, 2024 10:54:04 PM To: sumoDave/GarminDiveFitAppender @.> Cc: Chris Obermeier @.>; Author @.> Subject: EXTRe: [sumoDave/GarminDiveFitAppender] Error trying to merge 3 .fit files - (Issue #4)

Thanks for attaching the files . Seems that Order of Dive Summary messages have changed in fit files generated by the MK3. Will have a fix in soon.

— Reply to this email directly, view it on GitHubhttps://github.com/sumoDave/GarminDiveFitAppender/issues/4#issuecomment-2172140374, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BETMB4GFRYDRMYNTC2ZBUMLZHZMVZAVCNFSM6AAAAABJNFL62GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZSGE2DAMZXGQ. You are receiving this because you authored the thread.Message ID: @.***>

sumoDave commented 4 months ago

Fixed in release v2.0.0