svenmeier / coxswain

Your rowing servant
http://play.google.com/store/apps/details?id=svenmeier.coxswain
Apache License 2.0
80 stars 21 forks source link

60 Minuten Intervalltraining ist im Ergebnis 1 Stunde 59 Minuten. #98

Closed Aldebaran911 closed 4 years ago

Aldebaran911 commented 4 years ago

Hallo, ich habe mir ein Intervalltraining mit 6 Mal 10 Minuten und verschiedenen Pulszielen erstellt. Heute erstmalig gerudert und nach Beendigung stehen in der Trainingsdatei 1 h 59 Minuten. Vielleicht ein Fehler im Code? Ist so auch in Google Fit exportiert worden. Grüße Jens

svenmeier commented 4 years ago

Hi Jens,

in der Trainigs-Übersicht in der App steht welche Zeit?

Kannst du eine TCX Datei exportieren und hier angehängen?

Danke

Aldebaran911 commented 4 years ago

Hallo Sven,

kann ich leider nicht, da ich die vorhandene Datei im Editor geöffnet habe und die Sekunden von 7xxx Sekunden auf 3600 geändert habe. Nach erneutem Import passte es dann halbwegs. Jetzt steht im Ergebnis 1:00:09. Werde voraussichtlich morgen oder übermorgen die Datei nochmals rudern. Wenn es dann wieder sein sollte, sende ich sie Dir.

Grüsse aus Verl Jens PS: Ansonsten wirklich eine tolle App, vielen Dank dafür. Verstehe nicht, daß Waterrower soetwas nicht hinbekommt und gar drei verschiedene Softwarevarianten für USB Anschluss, Com Modul und Smart Row verõffentlicht.

Sven Meier notifications@github.com schrieb am Mi., 5. Feb. 2020, 16:23:

Hi Jens,

kannst du eine TCX Datei exportieren und hier angehängen?

Danke

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/svenmeier/coxswain/issues/98?email_source=notifications&email_token=AOPEKIKLYJLN666WD2OLSXDRBLKVTA5CNFSM4KQM57H2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEK32BNA#issuecomment-582459572, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOPEKIPD7LWUCNL7TLY7A63RBLKVTANCNFSM4KQM57HQ .

Aldebaran911 commented 4 years ago

Hallo Sven,

anbei die Datei von heute.

Grüsse aus Verl

Jens

svenmeier commented 4 years ago

Die Datei ist nicht durchgekommen (auch per Mail nicht), kannst du sie bitte hier auf dem Issue anhängen? ("Attach files by dragging & dropping...") https://github.com/svenmeier/coxswain/issues/98)

Aldebaran911 commented 4 years ago

Also die Möglichkeit des Einfügens von Dateien finde ich nicht. Nur Bilder können hier eingefügt werden 🤔. Wo kann ich per Drag and Drop was einfügen?

Aldebaran911 commented 4 years ago

Screenshot_2020-02-06-10-26-11-26_790ce7c340f790b20678444fdee2bd1e Screenshot_2020-02-05-22-14-44-99_790ce7c340f790b20678444fdee2bd1e

svenmeier commented 4 years ago

Oh, danke, ich hatte nicht verstanden, dass die Anzeige in der App schon verkehrt ist.

Kannst du mir die TCX-Datei an sven[at]meiers.net schicken?

svenmeier commented 4 years ago

Oh, danke, ich hatte nicht verstanden, dass die Anzeige in der App schon verkehrt ist.

Kannst du mir die TCX-Datei an sven[at]meiers.net schicken?

Aldebaran911 commented 4 years ago

Aber die .tcx Datei bekomme ich hier nicht hinein...

svenmeier commented 4 years ago

Ruderst du über USB oder Bluetooth?

Aldebaran911 commented 4 years ago

Bluetooth

Aldebaran911 commented 4 years ago

Habe ich geschickt

svenmeier commented 4 years ago

Danke, wie ich mir gedacht habe: Es wurden korrekt ~60 minuten getrackt, aber dann hat das S4 Bluetooth-Module noch eine falsche Zeit (~120 minuten) geliefert - Coxswain schreibt das nur auf.

Ich musste schon einen Workaround einbauen, da das Bluetooth-Module manchmal die falsche Zeit liefert.

Aldebaran911 commented 4 years ago

Und kannst Du das Problem lösen?

Sven Meier notifications@github.com schrieb am Do., 6. Feb. 2020, 14:27:

Danke, wie ich mir gedacht habe: Es wurden korrekt ~60 minuten getrackt, aber dann hat das S4 Bluetooth-Module noch eine falsche Zeit (~120 minuten) geliefert - Coxswain schreibt das nur auf.

Ich musste schon einen Workaround einbauen, da das Bluetooth-Module manchmal die falsche Zeit liefert.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/svenmeier/coxswain/issues/98?email_source=notifications&email_token=AOPEKIMJGPUMLZRAE5KBMILRBQF4LA5CNFSM4KQM57H2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEK7G2DI#issuecomment-582905101, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOPEKIMT33OT5EWMQQMZJB3RBQF4LANCNFSM4KQM57HQ .

svenmeier commented 4 years ago

Klar, da muss ich mir ein Workaround einfallen lassen.

Kannst du mal Test-weise dein Training auf 6x9 Minuten ändern? Ich vermute dann tritt das Problem nicht auf.

Aldebaran911 commented 4 years ago

Ja mache ich🥴. Wird wahrscheinlich erst am Wochenende.

Sven Meier notifications@github.com schrieb am Do., 6. Feb. 2020, 17:26:

Klar, da muss ich mir ein Workaround einfallen lassen.

Kannst du mal Test-weise dein Training auf 6x9 Minuten ändern? Ich vermute dann tritt das Problem nicht auf.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/svenmeier/coxswain/issues/98?email_source=notifications&email_token=AOPEKIIU3MQVN56W2E2XFLDRBQ2ZRA5CNFSM4KQM57H2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEK722SI#issuecomment-582987081, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOPEKIJAZJHPMOPT7HHFJVLRBQ2ZRANCNFSM4KQM57HQ .

Aldebaran911 commented 4 years ago

Hallo Sven,

heute mit 6x9 min war alles OK. Wenn Du die Datei trotzdem haben möchtest, dann gib mir Bescheid.

Grüße Jens

Jens A. jens.adam911@gmail.com schrieb am Do., 6. Feb. 2020, 18:27:

Ja mache ich🥴. Wird wahrscheinlich erst am Wochenende.

Sven Meier notifications@github.com schrieb am Do., 6. Feb. 2020, 17:26:

Klar, da muss ich mir ein Workaround einfallen lassen.

Kannst du mal Test-weise dein Training auf 6x9 Minuten ändern? Ich vermute dann tritt das Problem nicht auf.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/svenmeier/coxswain/issues/98?email_source=notifications&email_token=AOPEKIIU3MQVN56W2E2XFLDRBQ2ZRA5CNFSM4KQM57H2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEK722SI#issuecomment-582987081, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOPEKIJAZJHPMOPT7HHFJVLRBQ2ZRANCNFSM4KQM57HQ .

svenmeier commented 4 years ago

In Version 6.8 habe ich den Workaround erweitert. Hoffentlich fixed das auch das Problem bei der Stundengrenze.

Aldebaran911 commented 4 years ago

Super, vielen Dank. Probiere ich spätestens übermorgen aus.

Sven Meier notifications@github.com schrieb am Di., 11. Feb. 2020, 20:36:

In Version 6.8 habe ich den Workaround erweitert. Hoffentlich fixed das auch das Problem bei der Stundengrenze.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/svenmeier/coxswain/issues/98?email_source=notifications&email_token=AOPEKIKW6TYPX3DXE7P53O3RCL43JA5CNFSM4KQM57H2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELNYSAY#issuecomment-584812803, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOPEKIMWBFPOSMV4XALWBALRCL43JANCNFSM4KQM57HQ .

Aldebaran911 commented 4 years ago

Hallo Sven, leider funktioniert es doch noch nicht. Heutige Datei anbei. Oder muss ich das Intervalltraining zunächst neu erstellen? Grüsse Jens

Jens A. jens.adam911@gmail.com schrieb am Mi., 12. Feb. 2020, 09:47:

Super, vielen Dank. Probiere ich spätestens übermorgen aus.

Sven Meier notifications@github.com schrieb am Di., 11. Feb. 2020, 20:36:

In Version 6.8 habe ich den Workaround erweitert. Hoffentlich fixed das auch das Problem bei der Stundengrenze.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/svenmeier/coxswain/issues/98?email_source=notifications&email_token=AOPEKIKW6TYPX3DXE7P53O3RCL43JA5CNFSM4KQM57H2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELNYSAY#issuecomment-584812803, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOPEKIMWBFPOSMV4XALWBALRCL43JANCNFSM4KQM57HQ .

svenmeier commented 4 years ago

Nein, du musst nichts neu erstellen.

Ich denke, ich werde auch mal über eine Stunden rudern, um das Problem nachvollziehen zu können.

vonloxley commented 4 years ago

Das ist, für einen Softwareentwickler, ein erstaunlicher Einsatz!

svenmeier commented 4 years ago

Stimmt, bei 60 Minuten rudern komme ich wirklich mal ins Schwitzen ;-)

Aldebaran911 commented 4 years ago

Ja, der Einsatz ist grandios. Hoffentlich kauft Waterrower Dir die Software eines Tages für 1,5 Mio. ab.... . Ob man 6 Minuten mit maximaler Intensität oder 60 Minuten mit angepasster Intensität rudert, die Suppe läuft immer. Mir ist noch aufgefallen, das nach Export und erneutem Import der TCX Datei die Kraftkurve im Diagramm auf null steht.

Sven Meier notifications@github.com schrieb am Do., 13. Feb. 2020, 13:12:

Stimmt, bei 60 Minuten rudern komme ich wirklich mal ins Schwitzen ;-)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/svenmeier/coxswain/issues/98?email_source=notifications&email_token=AOPEKIOOMNJQHRFITCMBP43RCU2LJA5CNFSM4KQM57H2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELUXTLI#issuecomment-585726381, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOPEKIMN3SKKKOMZCWMJYLLRCU2LJANCNFSM4KQM57HQ .

svenmeier commented 4 years ago

Ok, 60 Minuten später kann ich den Fehler bestätigen.

Die Zeit vom Comm module springt bei 60 Minuten erst kurz auf 159 Minuten (7140 Sekunden).

Da muss ich den Workaround noch ein bisschen anpassen. Ich werde das Problem auch mit den Entwicklern von Waterrower besprechen.

Aldebaran911 commented 4 years ago

Ja schön! lch habe jetzt immer die Datei im Editor geöffnet, die Sekunden auf 3600 gesetzt und dann in Coxswain importiert. Die Zeit wurde in Deiner App dann immer mit 1:00:09 angezeigt. Nur wie bereits geschrieben geht dabei die Kraftkurve im Diagramm verloren.

Sven Meier notifications@github.com schrieb am Do., 13. Feb. 2020, 21:48:

Ok, 60 Minuten später kann ich den Fehler bestätigen.

Die Zeit vom Comm module springt bei 60 Minuten erst kurz auf 159 Minuten (7140 Sekunden).

Da muss ich den Workaround noch ein bisschen anpassen. Ich werde das Problem auch mit den Entwicklern von Waterrower besprechen.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/svenmeier/coxswain/issues/98?email_source=notifications&email_token=AOPEKIJP6ZTC4XKD5TITEUDRCWWY7A5CNFSM4KQM57H2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELWRWOA#issuecomment-585964344, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOPEKIPFX5QV3YCPC2DDI5LRCWWY7ANCNFSM4KQM57HQ .

fatman3 commented 4 years ago

Hallo, Ich finde die App wirklich super! Hatte auch das Problem, dass beim Import der TCX Datei die Kraftkurve nicht mehr eingelesen wird. Ich habe den Sourcecode durchstöbert und bin dann daruaf gekommen, dass in "garmin/TCX2Workout.java" der Wert nicht eingelesen wird. Man kann dem einfach abhelfen, indem man nach Zeile 149 folgendes einfügt: snapshot.power.set(Integer.parseInt(navigator.getText("Watts")));

grafik

svenmeier commented 4 years ago

Hi @fatman3 , danke das Thema hatte ich total vergessen.

Du hast recht, dein Fix sorgt dafür, dass die Kraft auch wieder eingelesen wird. Ich werde das für die nächste Version übernehmen.

Das ursprüngliche Problem mit der Zeitmessung nach einer Stunde ist bereit durch einen Workaround behoben.

Sven