search

Interrupt / delverengine

Delver game engine and editor
zlib License
803 stars 79 forks source link

Renaming example texture atlases to not collide with the base Delver assets #258

Closed Interrupt closed 3 years ago

Interrupt commented 3 years ago

Including the base Delver game assets as a mod for the Delver engine now mostly works. Makes testing changes to the engine with the stock assets much easier.

Also had to clean up the test levels to use the new atlas, and since I couldn't leave well enough alone I expanded out the sub levels test.

PythooonUser commented 3 years ago

We have some additional hard-coded texture atlas references. Should we also update these?

Dungeoneer/src/com/interrupt/dungeoneer/gfx/Material.java jsonschema/current/dungeoneer/gfx/Material.schema.json Dungeoneer/src/com/interrupt/dungeoneer/generator/rooms/HazardRoom.java Dungeoneer/src/com/interrupt/dungeoneer/generator/stairs/StairGenerator.java Dungeoneer/src/com/interrupt/dungeoneer/game/terrain/TerrainNoise.java Dungeoneer/src/com/interrupt/dungeoneer/generator/rooms/CaveRoom.java

Interrupt commented 3 years ago

Those hard coded texture atlases there are data driven and are meant to be overridden when setting up a level generator, so those are pretty safe.

A bigger problem is that we have some hard coded texture files in the code, namely textures.png. Without a default texture file named that, the game or editor will crash on startup. Maybe we should include a fallback warning texture that gets used when nothing is found.