Skip to content

Resolve "Erweiterte Farbauswahl"

d.kilic requested to merge 5-erweiterte-farbauswahl into master

Farbauswahl Merge Request

Änderungen

  • Hinzufügen der Möglichkeit eine Farbe per Mausklick auf die entsprechende Mütze auszuwählen
    • Color Picker Button im Tab Reco klicken/checken
      • Shift+Click - Farbe setzen/ mit dieser Farbe beginnen
      • Click - momentanen Farbbereich so erweitern, das diese Farbe (+ Buffer) enthalten ist
  • Migration des Build-System von qmake zu Cmake
    • Ab jetzt persönliche Konfiguration nicht mehr in der Datei festgehalten, sondern lokal von jeder einzelnen Person festzulegen
    • Baut jetzt auch direkt, ohne das man eine eigene Konfig angeben muss (Standard funktioniert)
    • Großteil von PeTrack wird jetzt als statische Bibliothek gebaut, welche von PeTrack.exe und den Tests gelinked wird (Vermeidet doppeltes bauen)
    • Einbinden von "moc_***.cpp" am Ende von Q_OBJECT-Klassen notwendig geworden
  • Einbinden von Catch2 (Testing) und trompeloeil (Mocking) in das Projekt
    • trompeloeil wird momentan nicht genutzt, ist aber fertig konfiguriert und kann bei Bedarf einfach included werden
    • Tests für die neue Farbauswahl wurden mit Catch2 implementiert
    • Ausführen von Unit Tests in CI
  • Kleinere unabhängige Änderungen
    • CopyToQImage für den RGB fall vereinfacht
    • Deprecation Warnings von Qt beachtet
      • qSort -> std::sort
      • QMatrix -> QTransform
      • sprintf -> asprintf
    • Entfernen von using namespace std aus allen Headerdateien
      • Hätte beim Kompilieren mit C++17 Probleme gemacht (momentan nutzen wir C++14, den Default bei unserer Version von MinGW)
    • Leak in Petrack.cpp gestopft ( mScene )
    • convertTo(QColor::Hsv) gibt eine neue Farbe zurück, statt das Objekt selbst zu verändern
      • Ersetzt durch toHsv()

Zu Beachten

  • Neues CopyToQImage braucht Qt 5.14 (aktuell neueste Version)
  • Die verschiedenen Methoden zur Farbauswahl können momentan nicht gemeinsam genutzt werden

Closes #5 (closed)

Merge request reports