Setting tiles in maps on-the-fly (Pull Request?)

[nightm4re94]

^{This post has been migrated automatically from the old LITIENGINE forum (2018 - 2022).
Posted: 2020-04-01 21:31:03
User: escho [] (age: 943 days 🙌; posts: 6)}
If I understand properly, LITIengine has no capability yet to set map tiles on-the-fly.
If this is true, would the devs / community be interested in my attempt to do so & a pull request?
Note: I realize there is a serialization capability. I'm talking about direct on-the-fly native LITIengine map generation capabilities.
I'm going to write it anyway non-native, so I thought I could do a native win-win.
Because of the groundwork already laid out, it would be higher level layer of code that converts java-like programming interpretation of maps into the existing serialization. i.e

GeneratedMap floor1 = new GeneratedMap(height: 100, width: 100)
for (x ... )  
for (y ... )  
floor1.setTileAtPoint(point, tile)  
...

and maybe it's too bulky, but HashMap does work with Point2D keys. It's a miracle of java really.

[nightm4re94]

^{This post has been migrated automatically from the old LITIENGINE forum (2018 - 2022).
Posted: 2020-04-02 20:33:45
User: steffen [Admin 👑, Mod 🔧] (age: 1568 days 🔥; posts: 100)}
escho I think the code model for maps should already allow for this task, at least we have done it in the past.
Some sample code from an older test implementation:

// create a basic map
 TmxMap map = new TmxMap(MapOrientations.ORTHOGONAL);
 map.setRenderOrder(RenderOrder.RIGHT_DOWN);
 map.setTileWidth(16);
 map.setTileHeight(16);
 map.setWidth(100);
 map.setHeight(100);
 map.setTiledVersion("1.1.5");
 map.setName("random");
 Tileset tileset = new Tileset(Resources.tilesets().get("sample"));
 map.getTilesets().add(tileset);
 
 List<Tile> tiles = new ArrayList<>();
 for (int y = 0; y < 100; y++) {
 for (int x = 0; x < 100; x++) {
 int tile = 3;
 if (SOME CONDITION TO SET A DIFFERENT TILE) {
 tile = 4;
 }
 tiles.add(new Tile(tile));
 }
 }
 TileData data;
 try {
 data = new TileData(tiles, 100, 100, TileData.Encoding.CSV, TileData.Compression.NONE);
 data.setValue(TileData.encode(data));
 } catch (IOException e) {
 data = null;
 e.printStackTrace();
 }
 TileLayer ground = new TileLayer(data);
 ground.setRenderType(RenderType.GROUND);
 ground.setWidth(100);
 ground.setHeight(100);
 map.addLayer(ground);

We should maybe simplify and improve the existing API here but I don't see a real reason for a different implementation. Any addition/contribution here would be very welcome :)

[nightm4re94]

^{This post has been migrated automatically from the old LITIENGINE forum (2018 - 2022).
Posted: 2020-04-02 21:24:01
User: escho [] (age: 943 days 🙌; posts: 6)}
steffen Yep! That would be map generation! Deepest apologies for this.

[nightm4re94]

^{This post has been migrated automatically from the old LITIENGINE forum (2018 - 2022).
Posted: 2020-04-02 21:41:55
User: escho [] (age: 943 days 🙌; posts: 6)}
Hmm, steffen That instigation of TileData is package-private & can't be accessed outside of LITIengine. I must be doing something wrong here.

[nightm4re94]

^{This post has been migrated automatically from the old LITIENGINE forum (2018 - 2022).
Posted: 2020-04-02 22:13:42
User: escho [] (age: 943 days 🙌; posts: 6)}
I'm going to see if this will work:

[  
data = (TileData)TileData.class.getMethod("TileData", (new ArrayList<Tile>()).getClass(), Integer.class, Integer.class, TileData.Encoding.class, TileData.Compression.class).invoke(null, tiles, 100, 100, TileData.Encoding.CSV, TileData.Compression.NONE);  
]

[nightm4re94]

^{This post has been migrated automatically from the old LITIENGINE forum (2018 - 2022).
Posted: 2020-04-02 22:17:14
User: steffen [Admin 👑, Mod 🔧] (age: 1568 days 🔥; posts: 100)}
You are right, I also noticed that when answering on this post... I've already fixed this in the litiengine master
see 3455fa6

[nightm4re94]

^{This post has been migrated automatically from the old LITIENGINE forum (2018 - 2022).
Posted: 2020-04-02 22:23:43
User: escho [] (age: 943 days 🙌; posts: 6)}
Yay! Let's see if I have it in me to compile her

[nightm4re94]

^{This post has been migrated automatically from the old LITIENGINE forum (2018 - 2022).
Posted: 2020-04-03 19:02:02
User: escho [] (age: 943 days 🙌; posts: 6)}
I'm loading the map into the environment, complete with a tsx tileset and it's not rendering the images. I know you'll probably want code.
Edit: Debugging reveals all tiles are null in the drawTile() method. TileLayer never finish() them so I'm guessing that makes TileSetEntry null.
Edit 2: That's it.
Adding:
try { map.finish(Game.class.getClassLoader().getResource("tilesheet.tsx")); } catch (TmxException e) { e.printStackTrace(); }
Works as intended.. so far..
Edit3: I may be doing something wrong, but I submitted Issue #329 just in case (#329)

[nightm4re94]

^{This post has been migrated automatically from the old LITIENGINE forum (2018 - 2022).
Posted: 2020-04-04 18:55:51
User: steffen [Admin 👑, Mod 🔧] (age: 1568 days 🔥; posts: 100)}
I've written an API to better support map generation from code (see this ~~~~GitHub comment)

IMap generatedMap;
try (MapGenerator generator = Resources.maps().generate(MapOrientations.ORTHOGONAL, "mymap", 100, 100, 16, 16, Resources.tilesets().get("tileset.tsx"))) {
 tileLayer = generator.addTileLayer(RenderType.GROUND, (x, y) -> {
 if (x == y) {
 // draw a diagonal in another tile color
 return 2;
 }
 // fill the entire map with this tile
 return 1;
 });
 // set an explicit tile at a location
 tileLayer.setTile(10, 10, 3);
 // add a collision box to the map
 generator.add(new CollisionBox(0, 50, 100, 10));
 generatedMap = generator.getMap();
}

This API will be available with the next release of the engine ~~~~v0.4.21-alpha or you can check it out in the master branch.
I've also update the Gurk Nukem project with an ~~~~example on how to use this new API.
Thank you for your valuable input on this topic.