onPlayerGenerateMapTile

Hallo @red51,

nachdem ich jetzt die screenshot Funktion in meine Plugins eingebaut habe (DiscordPlugin / GlobalIntercom) bin ich gerade dabei die neue requestMapTileRaw Funktion für mein Map Plugin zu verwenden und bin noch nicht ganz zufrieden.

Mein erstes Problem: es fehlt noch das richtige Event um die Funktion zu verwenden. Aktuell habe ich diese prototyp code hier in Verwendung:

@EventMethod
	public void onPlayerEnterChunk(PlayerEnterChunkEvent event) {
		Player player = event.getPlayer();
		// String lang = player.getSystemLanguage();
		Vector3i coords = event.getNewChunkCoordinates();
		// player.sendTextMessage("You are now in " + coords.x + "x " + coords.y + "y " + coords.z + "z");
		int tileX = coords.x / 3;
		int tileY = coords.z / 3;
		final File destinationFile = new File(tileRoot + "mt_" + tileX + "_" + tileY);
		String compareHash = FileUtils.getMd5(destinationFile);
		// player.sendTextMessage("You are now in MatTile" + tileX + "x " + tileY + "y ");
		player.requestMapTileRaw(tileX, tileY, compareHash, (byte[] tile) -> {
			if (tile == null) {
				// player.sendTextMessage("no tile returned");
			} else {
				// String tileChecksum = ByteUtils.getMd5(tile);
				// player.sendTextMessage("tile has " + tile.length + " bytes, checksum=" + tileChecksum);
				FileUtils.writeBytesToFile(tile, destinationFile);
			}
		});
	}

Ist noch nicht ganz korrekt der code aber damit komm ich erstmal weiter.
Der Vergleich mit dem MD5 funktioniert soweit, nur kam mir der Gedanke das ja die ingame MAP nicht immer direkt aktualisiert wird, also manchmal muss man halt den refresh button drücken.
Wenn jetzt jemand mit einer alten ingame map kommt hat der natürlich einen anderen MD5 weil die Daten anders sind. Dies verhindert dann aber nicht das die alten Daten die neueren überschreiben. Hier müßte man dann das datum der Datei noch vergleichen können oder besser dieser Vorschlag:

Ein neues Event "onPlayerGenerateMapTile" das gefeuert wird wenn der client ein map tile rendert bzw. gerendert hat. dort müßten dann die x/y Koordinaten übergeben werden. Diese kann man dann in requestMapTileRaw wieder verwenden um das entsprechende tile anzufragen.
So könnte man gewährleisten das immer nur neue tiles übertragen werden und keine alten.

Oh das wäre cool, dann kann ich vielleicht diese Woche noch eine erste Version meines Plugins veröffentlichen.

Oh super dann werde ich das nachher wenn ich am PC bin direkt mal ausprobieren.

Funktioniert super, bin gerade mal durch unbekanntes Gebiet und habe in bekanntem Gebiet auf resresh gedrückt. Keine Einschränkungen bzgl. lag oder sowas bemerkt, auch wenn bei refresh gleich mehrere übertragen werden. Eis ist mir noch aufgefallen das auch vorher schon war. im Server-Log steht immer SERVER MAPTILE 0 0 UPLOAD FROM _name_ COMPLETED! also immer 0 0

@red51 nachdem ich bemerkte das von meinen Spielern, wenn sie neues Gebiet erkunden gesprenkelte maps zurück kommen (teile fehlen) habe ich es selbst getestet und es werden nicht alle erstellten map files übertragen.

Hier der code den ich verwende:

@EventMethod
	public void onPlayerGenerateMapTile(PlayerGenerateMapTileEvent event) {
		Player player = event.getPlayer();
		int tileX = event.getX();
		int tileY = event.getY();
		final File destinationFile = new File(tileRoot + "mt_" + tileX + "_" + tileY);
		String compareHash = FileUtils.getMd5(destinationFile);
		player.requestMapTileRaw(tileX, tileY, compareHash, (byte[] tile) -> {
			if (tile == null) {
				// player.sendTextMessage("no tile returned");
			} else {
				FileUtils.writeBytesToFile(tile, destinationFile);
			}
		});
	}

Jain lokal sind die tiles ja da, wenn ich z.b. in eine Richtung laufe habe ich den Effekt das 3 tiles lokal erstellt werden aber nur 1 wird übertragen.

Ok werde ich mal heute Abend prüfen falls ich's zeitlich schaffe, danke! Hatte auch schon überlegt ob ich die requests in eine art queue packen soll um die pro-player sequentiell abzurufen damit nicht mehrere tiles gleichzeitig zu lag führen (ist bisher nie passiert aber kann ja immer mal jemand ne schlechte Leitung haben)

ok, deine Vermutung wird stimmen, mit dem delay kommt alles an