Resolve "Kamera - Trackballkamera"

Closes #42 (closed)

added 👍 Feature 💻 Doing 🖥️ I/O labels

added 1 commit

• a8ef6654 - [#42 (closed)] add Trackball implementation

Compare with previous version

added 1 commit

• 79eebd1e - [#42 (closed)] add 360 degree pitch rotation

Compare with previous version

added 73 commits

• 79eebd1e...3668b30b - 71 commits from branch develop
• edba5695 - [#42 (closed)] merge branch 'develop' into 42-kamera-trackballkamera
• e30d8c8a - [#42 (closed)] update main files for trackball showcase

Compare with previous version

Ich würde sagen, dass der Trackball soweit funktioniert. Gerne kann reviewed werden :D

removed 💻 Doing label

added 🙏 Review label

marked this merge request as ready

added 2 commits

• b1167732 - [#42 (closed)] add documentation (unvollstaendig)
• 8fee845f - [#42 (closed)] change variable name to offset

Compare with previous version

marked this merge request as draft

added 1 commit

• 4d1a5c0d - [#42 (closed)][Fix] clamp yaw member in camera class ass well

Compare with previous version

added 1 commit

• 7de8b93d - [#42 (closed)][Fix] correct standard up vector rotation

Compare with previous version

added 3 commits

• 637e6a7b - [#42 (closed)][Doc] add documentation of camera class
• 399036df - [#42 (closed)][Doc] add documentation for CameraManager class
• cb9928ba - [#42 (closed)][Doc] add documentation to TrackballCamera class, adjust documentation of Camera

Compare with previous version

added 6 commits

• cb9928ba...ff713793 - 4 commits from branch develop
• db0f5304 - Merge branch 'develop' into 42-kamera-trackballkamera
• dab6bc9f - [#42 (closed)][Fix] mirror rotation around X axis

Compare with previous version

added 1 commit

• 9300e837 - [#42 (closed)] add vertical movement via E and Q keys to Camera

Compare with previous version

marked this merge request as ready

added 1 commit

• 9fed028f - [#42 (closed)] add const to getter and remove redundant function

Compare with previous version

Bei mir funktionieren keine Tasten als Eingabe mehr und alle Anwendungen verwenden die Trackball-Kamera... sollten nicht beide Kameras funktionieren?

Ich würde auch vorschlagen, dass m_top und m_bottom zu m_upwards und m_downwards oder m_up und m_down umbenannt werden. Bei der Bewegung der Kamera ist mir auch aufgefallen, dass der up-Vektor auf die Position addiert wird, wenn die Bewegung nach unten gehen soll. Ist das korrekt? Speicher ihr vielleicht nicht den up-Vektor sondern down?

Aktuell ja, das hatte mich auch verwundert. Mir wurde gesagt, dass Alex gemeint hätte, das wäre so eine Konvention in Vulkan. Ich hatte das dann erstmal so hingenommen. In unserem Vortrag hatten wir zumindest auch angemerkt, dass man die y-Scaling-Kompontene flippen muss, da sonst das Objekt auf dem Kopf steht. Ich schätze, dass ist das, was wir noch machen müssen. Dann können wir wahrscheinlich auch wieder den normalen Up Vektor nutzen