search

Mastbau-FN / inspector

fullstack Projekt zur Vereinfachung von Inspektionen
https://mastbau-fn.github.io/inspector/app/
3 stars 1 forks source link

Übertragungsproblem beim Arbeiten mit mehreren Inspektionen #343

Open MBG-UN opened 1 year ago

MBG-UN commented 1 year ago

(der Text aus der Mail) Der Rücklauf aus der ersten richtigen Feldeinsatz war Inspektions-App bezogen richtig positiv. Die Kollegen haben alles vor Ort mit der App richtig gut erledigen können.

Leider gabs beim Zurückspielen der Daten eine Problem. Vielleicht liegt die Ursache in der Offline-Bearbeitung mit der hohen Zahl der offlinegestellten Inspektionen zusammen. Es waren ca. 10 Inspektionen, die nacheinander offline bearbeitet worden sind (ohne zwischenzeitlicher Synchronisation mit dem Server)

Die mit einer positiven Meldung abgeschlossene Synchronisierung hat auch gute 2,5 Stunden benötigt. Ein Indiez, dass alle INfos von allen Inspektionen auch übertragen wurden.

Beim Berichterstellen in der MBG DB fielen dann aber doch Datenlücken auf. Momentan mindestens in einer Inspektion, bei der offline angelegte Mängel und Kategorien mit ihren Prüfpunkten nicht in der MBG DB angekommen sind. Dabei war auffällig, dass dies immer beides betraf: sowohl inhaltliche Daten (Tabellen-Daten) als auch die dazugehörenden Bilder. Eine genaue Untersuchung des Datenbestand auf dem Server ist nicht mehr möglich, da nach erfolgreicher Übertagung in dei MBG DB die Daten auf dem Server komplett gelöscht werden. Die betroffene Inspektion ist jedoch noch im Original auf dem Tablet erhalten. Da nach dem Einspielen der betroffenen Inspektion keine Synchronisation mehr zwischen App und Server stattgefunden hat und wir auch das Tablett nicht im WLAN oder Datenverbindungs-Modus laufen haben. Beim nächsten "Refresh" würde sonst diese für das Analysieren der Fehlersituation notwendigen Daten löschen.

Da wir ungern das Tablet für die Fehleranalyse online stellen wollen, müssten wir uns vielleicht für Montag oder Dienstag im Büro treffen. Bevor wir die Fehleranalyse starten, müssen wir die in der App abgelegten Bilder retten, da dieses nur dort gespeichert sind. Ich habe sie nirgends auf dem Tablet gefunden. Ihr habt da garaniert eine versteckten Modus, mit dem man das klären könnte.

Eine weitere Fehlerursache könnte das Herunterladen einer weiteren Inspektion gewesen sein. Dazu sollten wir uns aber unterhalten.

Wann könntest Du/Ihr Montag (besser) oder Dienstag.?

HannesGitH commented 1 year ago

Ja, an die Daten der App kommt man nicht ohne weiteres ran, das soll auch so, damit keine anderen bösen Apps oÄ an die Daten kommen.

Zum retten der daten gibt es folgende (nicht exklusive) Möglichkeiten, welche alle ein Updaten der App benötigen:

push debug build

da ihr gerade ein release-build der App verwendet, kann man die Daten auch nicht mit PC extrahieren. man könnte ein update pushen welches das release build durch ein debug build ersetzt, dann können wir am PC die Dateien anschauen und extrahieren

dazu muss in der build.gradle der debug key durch den release key ersetzt werden damit das update von android ohne datenverlusst angenommen wird:

debug {
      signingConfig signingConfigs.release
    }

das sollte die schnellste, kurzfristigste Lösung sein

Data sharing function

alternativ könnte man ein update pushen mit dem eine extracktion der daten ermöglich wird, einerseits einzelner bilder (https://github.com/Mastbau-FN/inspector/blob/96ca60c485961cacb3b086887f56fc85393ee65b/frontend/lib/fragments/imageWrap.dart#L32) aber auch ganzer Datenpunkte (oder dem kompletten data-directory zum debuggen) als .zip

remote debugging package

als letzte option könnte man ein remote debugging tool (siehe #344) einbauen um auch zukünftig solche Situationen besser diagnostizieren zu können.

ohne update können wir es vorher mal mit adb backup -noapk com.mbgsolutions.inspector risikofrei probieren

bis morgen ;)

HannesGitH commented 1 year ago

backup läuft

die failed_requests sind auch noch relativ voll, was sehr gut ist, weil dann wahrscheinlich einfach nochmal der upload/sync knopf gedrückt werden muss/kann

HannesGitH commented 1 year ago

die failed_requests sind auch noch relativ voll, was sehr gut ist, weil dann wahrscheinlich einfach nochmal der upload/sync knopf gedrückt werden muss/kann

sieht ganz danach aus :D upload läuft jetzt

(closed as solved for now, reopen if necessary)

HannesGitH commented 1 year ago

upload läuft jetzt

nur so halb

HannesGitH commented 1 year ago 
SELECT "Link" , "LinkOrdner" -- Ordner für Fotos
FROM "Events"
WHERE (
    "EventID" = $2
    AND "PjNr" = $1
)
;

gibt nix für EventID 1910 und PjNr 20236169

und daran scheitert der upload, dort sollte hintelegt sein, wo die bilder für diesen standort hinterlegt werden sollten

HannesGitH commented 1 year ago

@MBG-UN

MBG-UN commented 1 year ago

=> "gibt nix für EventID 1910 und PjNr 20236169" Da kann es auch ncihts geben, da diese Inspektion schon eingecheckt ist. Danach wird alles auf Ubunte-Server gelöscht. Oder fehlte das lokal auf dem Tablet?

Bitte 20236161 ausprobieren, Da traten die ersten Fehler auf. Danach hat Pratik aufgehört, die Inspektionen einzuchecken. Hier könnte auch schon alles gelöscht sein. Besser ist 20236163 . Hier fehlen nur wenige Punkte. Bzw 20236162 oder 20236170 . Hier fehlt alles.

MBG-UN commented 1 year ago

Image

MBG-UN commented 1 year ago

Image

HannesGitH commented 1 year ago

gibt nix für EventID 1910 und PjNr 20236169 Da kann es auch ncihts geben, da diese Inspektion schon eingecheckt ist. Danach wird alles auf Ubunte-Server gelöscht. Oder fehlte das lokal auf dem Tablet?

nein, fehlte nichts auf dem tablet, aber einige Änderungen wurden einfach nicht hochgeladen, diese sind zeitlich sortiert, d.h. der erste erfolglose request wird als erstes ausprobiert. Dieser schlägt aber fehl, weil die oben erwähnte sql-query im backend nichts zurückgibt.

Ich weiß also nicht ob diese Inspektion tatsächlich schon hätte eingecheckt werden sollen, oder ob da nicht noch etwas fehlt.. wenn du aber sagst das passt so schon dann können wir alle requests zu diesem standort verwerfen und mit dem ersten nicht eingecheckten fortfahren. *

Bitte 20236161 ausprobieren, Da traten die ersten Fehler auf. Danach hat Pratik aufgehört, die Inspektionen einzuchecken. Hier könnte auch schon alles gelöscht sein. Besser ist 20236163 . Hier fehlen nur wenige Punkte. Bzw 20236162 oder 20236170 . Hier fehlt alles.

*: das würde schon gehen (aber etwas aufwändiger ein suchen nach dem ersten auszuprobierenden request mitbringen) und die anderen requests haben wir ja immernoch im backup falls man diese später noch ausführen will

HannesGitH commented 1 year ago

garantiert muss aber die meldung angepasst werden, dass klar ist, dass der upload fehlschlug

MBG-UN commented 1 year ago

Inspektion 20236169 ist definitv eingecheckt. Ich bin sie eben durchgegangen und habe kleine Fehler gefixt, die aber nichts mit dem Synch-Abbruch zusammenhängen. (Nicht aufgelöster Bilder-Link durch Leerzeichen-Mapping auf %20) Bitte trotzdem noch nicht löschen. Da muss erst Pratik nochmal drüber schauen. Also leider bis morgen warten.

HannesGitH commented 1 year ago

ja wenn dann würde auch nur ein bild fehlen, aber das existiert auch nicht also sollte passen.

in 20236166-6-2 sollte ein 'ohne mangel mit "Fußplatte- Eine schraube korrodiert" fehlen?

MBG-UN commented 1 year ago

Inspektion 20236169 ist definitv eingecheckt und übertragen. Ich bin sie eben durchgegangen. Habe sonst keine Fehler gefunden. Du kannst sie löschen.

MBG-UN commented 1 year ago

In Inspektion 20236166-6-2 gibt es 3 Mängel bei 6.2 Konstruktion Der markierte Mangel 6.2.3 wurde wohl von "ohne Mangel" auf "Mangel leicht" umdefiiniert.

Image

HannesGitH commented 1 year ago

das (6.2.3) ist der mit LangText "Fußplatte- Eine schraube korrodiert" ? wenn ja, hat der auch drei bilder?

(locally_addedCAP5969405499447178266.jpg, locally_addedCAP6813426409341157976.jpg, __locally_added__CAP2208686641964270201.jpg)

MBG-UN commented 1 year ago

das wird ein nicht übertragener Mangel sein. Weder Text noch JPEG-Namen sind bei uns vorhanden. Das müsste ich am Freitag früh mit Pratik klären.

MBG-UN commented 1 year ago

6.2.3 ist scheinbar doppelt. Vielleicht ein Puntk der von Pratik OFFLINE überarbeitet wurde? Auch das versuche ich mit Pratik am Fr. zu klären.

chrisoesterreichprog commented 1 year ago

related to Issues Prio > 5