Dungine: remove libGDX and improve the ECS approach (thesis/concept study) #1652
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Hab die Dateien resettet. Das die Imports geändert sind, liegt an einer Einstellung in IntelliJ, wie Imports geschrieben werden. Die Standardeinstellung fasst ab einer definierten Anzahl an Imports aus dem selben Package diese zu |
Danke Dir :)
Hmmm, wusste gar nicht, dass man über die EditorConfig darauf Einfluss nehmen kann? Hatte eher Richtung Checkstyle oder (besser) Spotless vermutet? |
|
@fwatermann Hmmm, es compiliert jetzt :) Aber Spotless und Checkstyle sind beide noch nicht happy. Kannst Du bitte noch einmal Spotless über Deinen Code laufen lassen? Bei Checkstyle fehlt bestimmt nur irgendwo noch ein Javadoc, oder die erste Zeile entspricht vielleicht noch nicht den Konventionen (muss mit einem "." enden)? Edit: Hab grad in den Log geschaut. Spotless sollte easy sein (einmal |
|
checks sind grün 🎉🥳 |
|
@fwatermann Ich hab noch eine Idee. In Deiner Abschlussarbeit beschreibst Du ja die verschiedenen Teile der Dungine und vergleichst das mit dem ursprünglichen Dungeon-Projekt. Ich könnte mir das gut als Dokumentation unter Was denkst Du? Afaik hattest Du ja mit dem Markdown-Template gearbeitet, oder? Da könntest Du das direkt rüberkopieren und das Hauptdokument nennst Du einfach |
|
@cagix hast du das auf Mac zum laufen bekommen? Ich scheitere an OpenGL |
Das Problem sollte gefixt sein |
@fwatermann @AMatutat Ich kriege es zumindest gebaut. Bei 
Aber bei jedem Button (bis auf "Exit" :) stürzt das Mopped dann ab. Bei "Beleuchtung":Bei "CameraComponent":Bei "Erde, Mond & Sonne":Bei "PlayerComponent":Bei "3D-Level":Hmmm. Es geht i.d.R. bei Ich werde das Gefühl nicht los, dass hier irgendwas noch fehlt bzw. noch nicht plattformspezifisch runtergeladen wird? |
@cagix Die beiden Fehler sollten gefixt sein. Habe nur leider keinen Mac zum testen da :/ |
|
@fwatermann du bist echt schnell! bist du sicher, dass du nicht im dungeon mal was machen willst? ich bin grad komplett unter wasser, hoffentlich kann ich das heute nochmal starten. |
hmmm. gefixt insofern, dass jetzt in allem items die jvm hart abstürzt 🤣 das letzte, was vor dem absturz kommt, war edit: ich hab mal meine gradle-caches geleert (lokal im projekt, global aufm rechner) und nochmal neu gebaut - selbes ergebnis. |
cagix
requested changes
Nov 18, 2024
There was a problem hiding this comment.
Irgendwas klemmt auf macOS (ARM) mit der geladenen Bibliothek - die Java-VM stürzt hart ab. Entweder muss noch was an der Gradle-Konfiguration angepasst werden oder die Lib tut einfach nicht auf ARM? Dem muss noch nachgegangen werden (@cagix, @AMatutat).
Wir haben bei der Demo verabredet, dass im Doc-Ordner noch die relevanten Teile der BA als Markdown reinkommen (@fwatermann).
@fwatermann Du kannst die Literatur-Referenzen mit Pandoc auflösen:
pandoc -s -t gfm -f markdown
-M lang=de-DE --metadata-file=md/metadata.yml
--citeproc -M bibliography=references.bib -M link-citations=true
md/INPUT1.md md/INPUT2.md md/INPUT3.md md/INPUT4.md
-o OUTPUT.md
... ggf. möchtest Du als letzte Überschrift noch ein ## Quellen einfügen (das Verzeichnis wird nämlich einfach ans Ende des Dokuments gepackt).
|
Mist. Auf einem Intel-Mac kommt nicht mal das Auswahlfenster hoch. Das Ding geht sofort mit einem java.lang.UnsatisfiedLinkError sterben ... |
cagix
changed the title
cagix
reviewed
Dec 1, 2024
There was a problem hiding this comment.
@AMatutat Ich kriege das weder unter Intel-Mac noch M1-Mac zum Laufen. Das scheint aber vor allem ein Konfigurationsproblem zu sein, der Code läuft aber tatsächlich, zumindest unter Windows (@fwatermann hatte @bcg7 und mir nochmal ne Demo gezeigt). @bcg7 wollte eigentlich auch nochmal schauen, ob es auch unter Linux tut, aber vermutlich hat sie keine Zeit.
Ich finde das Ergebnis trotzdem sehr spannend und würde es gern zumindest als Konzeptstudie mergen. Da sind neben dem direkten Einsatz von LWJGL/Verzicht auf libGDX viele interessante Ideen drin, die den Code im "alten" Dungeon deutlich verbessern könnten.
Hmm okay intel-mac hätte ich jetzt eigentlich erwartet....
Gibts Fotos oder so? Hab das noch gar nicht gesehen |
Leider nein, auch kein Screencast. Sieht erstmal so aus wie unser Dungeon (selbes Tile-Set), aber Du kannst halt beliebig hoch und runter ... und da, wo die Tiles nicht hinkommen, sieht es aus wie das Holodeck vor dem Start 😃 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
To be filled...