Skip to content

Blockly: add demo level for dfs and bfs#1940

Merged
AMatutat merged 23 commits intomasterfrom
tobi511/ki-level
May 6, 2025
Merged

Blockly: add demo level for dfs and bfs#1940
AMatutat merged 23 commits intomasterfrom
tobi511/ki-level

Conversation

@Tobi511
Copy link
Copy Markdown
Contributor

@Tobi511 Tobi511 commented Apr 30, 2025 •
edited
Loading

Basiert zum Teil auf #1924 und sollte erst danach gemergt werden.

Es werden drei neue Level hinzugefügt:

  • dfs_1: Dieses Labyrinth ist mit der Tiefensuche (1 Min 13 Sek) schneller zu lösen als mit der Breitensuche (2 Min 30 Sek).
  • bfs_1: Dieses Labyrinth ist mit der Breitensuche (1 Min 11 Sek) schneller zu lösen als mit der Tiefensuche (1 Min 40 Sek).
  • leveltemplate_1: Dies ist eine leere Vorlage zum Erstellen eines eigenen Labyrinths.

Außerdem wird ein neuer Starter ClientKiStarter hinzugefügt, damit diese neuen Level korrekt angezeigt und verwendet werden können.

Zusätzlich habe ich das LevelEditorSystem und den DungeonSaver einmal ins Blockly rüberkopiert.

Closes #1855 und #1862 bedingt.

@Tobi511 Tobi511 self-assigned this Apr 30, 2025
@Tobi511 Tobi511 marked this pull request as ready for review April 30, 2025 10:28
@cagix
Copy link
Copy Markdown
Member

cagix commented Apr 30, 2025

@Tobi511 Kann es sein, dass dieser PR auf @Flamtky's #1924 basiert, weil hier wie dort das PathfindingSystem eingeführt wird (und alle anderen Pathfinding-Klassen auch)?

Falls das so ist, schreibe bitte in den ersten Kommentar deutlich rein, dass erstmal #1924 gemergt werden muss und dieser PR hier rebased werden muss ... und setze diese PR bitte am besten auf "Draft" für die Zwischenzeit.

@cagix cagix changed the title Ki Demo Level Blockly: add demo level for dfs and bfs Apr 30, 2025
@Tobi511
Copy link
Copy Markdown
Contributor Author

Tobi511 commented Apr 30, 2025

Ja, stimmt, da war ich etwas zu voreilig. @Flamtky hatte mir schon mal ein paar Zwischenstände rübergeschoben, damit ich die Level entsprechend testen und anpassen kann.

@Tobi511 Tobi511 marked this pull request as draft April 30, 2025 11:20
@Tobi511 Tobi511 marked this pull request as ready for review May 5, 2025 10:01
@Flamtky Flamtky force-pushed the tobi511/ki-level branch from 3deed97 to 3f612fc Compare May 5, 2025 10:27
Flamtky
Flamtky previously requested changes May 5, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@Flamtky Flamtky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nur 1-2 kleine Schönheitsfehler

Comment thread blockly/src/starter/ClientKiStarter.java Outdated
Comment thread blockly/src/starter/ClientKiStarter.java Outdated
Comment thread blockly/src/starter/ClientKiStarter.java
Tobi511 and others added 2 commits May 5, 2025 12:57
@Tobi511 @Flamtky
Update blockly/src/starter/ClientKiStarter.java
89b9a8e 
Co-authored-by: Niklas Schumann <68606032+Flamtky@users.noreply.github.com>
@Tobi511
change javadoc and name of ki starter
fccf792
@Tobi511 Tobi511 added the blockly label May 5, 2025
@AMatutat
Copy link
Copy Markdown
Contributor

AMatutat commented May 5, 2025

@Tobi511 kannst du drei Screenshots schicken? Dann seh ich direkt wie die level aussehen

AMatutat
AMatutat reviewed May 5, 2025
View reviewed changes
Comment thread blockly/src/starter/KiStarter.java Outdated
AMatutat
AMatutat reviewed May 5, 2025
View reviewed changes
Comment thread blockly/src/systems/LevelEditorSystem.java
AMatutat
AMatutat reviewed May 5, 2025
View reviewed changes
Comment thread blockly/src/utils/DungeonSaver.java
Tobi511 added 2 commits May 5, 2025 14:29
@Tobi511
move DevDungeonSaver and LevelEditorSystem to dungeon/src/contrib, ch…
4777b77 
…ange ki level theme to default and edit pathfinder starter filename
@Tobi511
apply spotless
2a67cbc
AMatutat
AMatutat reviewed May 5, 2025
View reviewed changes
Comment thread dungeon/src/contrib/level/DevDungeonSaver.java
AMatutat
AMatutat reviewed May 5, 2025
View reviewed changes
Comment thread dungeon/src/contrib/systems/LevelEditorSystem.java
@AMatutat AMatutat self-assigned this May 6, 2025
AMatutat
AMatutat previously approved these changes May 6, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@AMatutat AMatutat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@AMatutat AMatutat requested a review from Flamtky May 6, 2025 06:16
@AMatutat
Copy link
Copy Markdown
Contributor

AMatutat commented May 6, 2025

@Flamtky weil ich den letzten commit gemacht hab müsstest du einmal nicken

cagix
cagix previously approved these changes May 6, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@cagix cagix left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AMatutat Zwei Drei Sachen. Kannst Dir überlegen, ob Du dafür Tickets aufmachst und die Threads schliesst und dann mergst, oder ob Du die Sachen noch adressieren willst.
(Deshalb "approve", aber die Kommentare verhindern den Merge :)

Comment thread blockly/src/starter/PathfinderStarter.java Outdated
Comment thread blockly/src/starter/PathfinderStarter.java Outdated
cagix
cagix reviewed May 6, 2025
View reviewed changes
Comment thread blockly/src/starter/PathfinderStarter.java Outdated
@AMatutat AMatutat dismissed stale reviews from cagix and themself via cdb310f May 6, 2025 09:08
@AMatutat AMatutat requested a review from cagix May 6, 2025 09:09
@cagix cagix mentioned this pull request May 6, 2025
Closed
cagix
cagix approved these changes May 6, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@cagix cagix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@cagix cagix dismissed Flamtky’s stale review May 6, 2025 09:22

die punkte aus @Flamtky's review scheinen bearbeitet zu sein?

@cagix
Copy link
Copy Markdown
Member

cagix commented May 6, 2025

die codeowner s*cks.

@Flamtky du musst wohl leider auch nochmal drauf schauen :)

AMatutat
AMatutat approved these changes May 6, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@AMatutat AMatutat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hab ich gut gemacht

@AMatutat AMatutat merged commit db9d380 into master May 6, 2025
9 checks passed
@AMatutat AMatutat deleted the tobi511/ki-level branch May 6, 2025 09:23
@AMatutat
Copy link
Copy Markdown
Contributor

AMatutat commented May 6, 2025

die codeowner s*cks.

@Flamtky du musst wohl leider auch nochmal drauf schauen :)

oder auch nicht :P

Versteh einer Git.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cagix cagix cagix approved these changes

@AMatutat AMatutat AMatutat approved these changes

@Flamtky Flamtky Awaiting requested review from Flamtky

Assignees

@AMatutat AMatutat

@Tobi511 Tobi511

Labels

blockly

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

KI Demo-Level

4 participants

@Tobi511 @cagix @AMatutat @Flamtky