Diskussion:SDE Systementwurf SoSe2025: Autonome Fahrbahnvermessung: Unterschied zwischen den Versionen
Zur Navigation springen
Zur Suche springen
| Zeile 62: | Zeile 62: | ||
= Code Review = | = Code Review = | ||
* Wo finde ich das PAP für <code>weisselinie_folger.ino</code> | * Wo finde ich das PAP für <code>weisselinie_folger.ino</code> | ||
* Ich finde mich in Ihrem Ordner <code>https://svn.hshl.de/svn/MTR_SDE_Praktikum/trunk/_Semesterordner/SS2025/Sprint_1</code> nicht zurecht. | |||
{| class="wikitable" | {| class="wikitable" | ||
Version vom 28. April 2025, 14:32 Uhr
--Ulrich.schneider@hshl.de (Diskussion) 19:09, 14. Apr. 2025 (CEST)
Status Tag vor Termin 4/4
- Bitte dokumentieren Sie fortlaufend. Im Artikel wurde noch nicht gearbeitet.
- Die Schritte 1-12 wurden bereits erledigt, jedoch nicht dokumentiert.
- Bereiten Sie den Meilensteintermin am 29.04.25 um 08:15 Uhr vor. Entnehmen Sie dem Seminarvortrag "Scrum", was zum Sprint gehört.
--Ulrich.schneider@hshl.de (Diskussion) 13:37, 28. Apr. 2025 (CEST)
- Abb. 1 ist sehr anschaulich. Verlinken Sie jede Abbildung im Text.
- Abb. 2: Ich würde das Koordinatensystem noch transformieren, so dass alle xy-Koordinaten auf der Fahrbahn stets positiv sind.
- Wieso kommt Tabelle 2 vor 1?
- Der Text verweist auf Abb. 6 und Abb. 7. Wo finde ich die?
- Beschriften Sie alle Tabellen oben.
- Tabelle 2: linksbündig beschriften
- Klappen Sie die Terminübersicht ein.
- Die Abhängigkeiten im Gantt-Diagramm sind nocht vollständig dargestellt. Der Meilenstein am 29.04.25 taucht nicht auf.
- Am 15.04.25 wollten Sie die Mittellinie darstellen. Sie ist jedoch nicht daregestellt. Wieso?
- Die Planungsdaten (P) sind unvollständig.
- Die Umsetzung (D) wurde nicht dokumentiert. Wo haben Sie die TopCon Station plaziert?
- Tests (C) wurden nicht dokumentiert.
- Ergebnisse (A) wurden nicht dokumentiert.
Es fehlen folgende Abschnitte/Beschreibungen:
- CAD-Design der Montageplatte für den AlphaBot
- Review und 3D-Druck durch Merrn Ebmeyer
- Montage der Prismabefestigung
- Inbetriebnahme AlphaBot Linienverfolger für durchgezogene Linie
- Einarbeitung und Inbetriebnahme der Referenzstation
- Testweise Erstellung einer Referenzmessung
- Referenzmessung für rechte und linke Fahrspur
- Optional: Anpassung der Software für die Mittelspur ODER geometrische Berechnung der Mittelspur
- Bereinigung von Fehlern in der Karte
- Ablage der digitalen Karte als
.mat-Datei. - Messung einer Roboterbewegung und Anzeige auf der Karte.
- Test der Anforderungen und Dokumentation
- ggf. Überarbeitung/Verbesserung des Artikels Referenzmessung_mit_der_Topcon_Robotic_Total_Station
- Analyse und Bewertung der Ergebnisse im Wiki-Artikel
| Req. | Beschreibung | Priorität | Bemerkung |
|---|---|---|---|
| 1 | Ein AlphaBot muss nacheinander die drei Linien der Fahrbahn autonom verfolgen. | 1 | 2/3 |
| 2 | Als Referenzmessystem kommt die Topcon Robotic Total Station zum Einsatz. | 1 | ✓ |
| 3 | Das Prisma wird auf einer zu fertigenden Halterung über der Lenkachse des AlphaBot montiert. | 1 | ✓ |
| 4 | Die Referenzwerte müssen mit MATLAB® aufgezeichnet werden. | 1 | x Ergebnislink fehlt |
| 5 | Fehler in den Messwerten müssen bereinigt werden. | 1 | x |
| 6 | Die zweidimensionale digitalen Karte muss als MATLAB®-Datei (.mat) bereitgestellt werden. |
1 | x Ergebnislink fehlt |
| 7 | Das Vorgehen muss als Gantt-Diagramm geplant werden. | 1 | x Kommentare siehe oben |
| 8 | Lösungsweg und Lösung muss im Wiki dokumentiert werden. | 1 | x Kommentare siehe oben |
| 9 | Nach Erstellen der digitalen Karte muss die Fahrt des AlphaBot in der Karte eingezeichnet werden. | 1 | x Ergebnislink fehlt |
Code Review
- Wo finde ich das PAP für
weisselinie_folger.ino - Ich finde mich in Ihrem Ordner
https://svn.hshl.de/svn/MTR_SDE_Praktikum/trunk/_Semesterordner/SS2025/Sprint_1nicht zurecht.
| Zeile | Bemerkung | |
|---|---|---|
| 4 | Modulname stimmt nicht | |
| 11 | R | |
| 15 | Version stimmt nicht. | |
| 25 | Datum stimmt nicht. | |
| 38 | Wieso Alf? | |
| 42-49 | Vermeidnen Sie globale Variablen. | |
| 43 | Einheitliche Dateilänge verhindert Umrechnungsfehler. | |
| 45, 46 | Postfix fehlt | |
| 54 | Vermeiden Sie Magic Numbers. | |
| 65, 66 | Postfix fehlt | |
| 70 | Kommentar fehlt | |
| 87 | Postfix fehlt, Overflow? | |
| 92 | Falsche Typenzuweisung ohne Typecast | |
| 96 | Rechnen Sie in float | |
| 106-110 | Wieso begrenzen Sie nicht die Motorstellgröße? | |
| 114, 115 | Diese Regelung ist seltsam. Wieso steuern Sie beide Räder an? Werte kleiner 80 kann der AlphaBot häufig nicht umsetzen. | |
| 118, 119 | Diese Variablen werden nicht verwendet. | |
| 123 | Müssen Sie immer neu kalibrieren? Sind die Werte für den Raum konstant? |