Expanding & Overviewing the map
#11
OK. Tracked it down. It comes down to our storage format.
There is no modification time set for our chunks, and the overviewer rejects any chunks without an mtime.
Reply
Thanks given by:
#12
That should be fairly easy to fix - whenever storing a chunk, store its modification time ("time(nullptr)") as well.
Reply
Thanks given by:
#13
(11-29-2014, 07:41 AM)xoft Wrote: That should be fairly easy to fix - whenever storing a chunk, store its modification time ("time(nullptr)") as well.

Yup. Need another coder? I don't mind adding to this, but I don't want to step on any toes.
Reply
Thanks given by:
#14
Are you kidding? We always welcome new people willing to donate code. Feel free to submit a PR on GitHub with this change.
Reply
Thanks given by:
#15
Checking out the Anvil storage mechanism over some turkey soup today. Smile
Reply
Thanks given by:
#16
Well, I have the mtime stored now, but it looks like like the ChunkWorx plugin doesn't quite generate everything.

Areas I have acutally been in show as lit up, areas generated by ChunkWorx are dark.

http://minecraft.planetx.com/
Reply
Thanks given by:
#17
Right, ChunkWorx doesn't light the chunks, it only generates the terrain. We need to do some work on this. The plugin is ripe for a rewrite anyway.
Reply
Thanks given by:
#18
Plan of action:
- Implement a cWorld::PrepareChunk() method that either loads the chunk from disk or generates it, as needed, and lights it.
- (Use that function in cSpawnPrepare for cWorld - https://github.com/mc-server/MCServer/issues/1615 )
- Export the function to Lua
- Change the ChunkWorx plugin to use the function instead of cWorld:TouchChunk()
- Change the ChunkWorx plugin to use a better queueing and smarter chunk ordering.
Reply
Thanks given by:
#19
I'll submit the first needed changes to WSSAnvil.cpp/h today or tomorrow. (It's barely 2 dozen lines of code.)

As to lua, this is my first experience. I'm a VLSI and hardware drivers guy -- mainly for crypto. I'll have a look at cWorld and what you are doing there this weekend.
Reply
Thanks given by:
#20
(11-30-2014, 05:18 AM)xoft Wrote: Are you kidding? We always welcome new people willing to donate code. Feel free to submit a PR on GitHub with this change.

The create PR button is disabled for me.

Here are the two files.


Attached Files
.cpp   WSSAnvil.cpp (Size: 78.24 KB / Downloads: 2,105)
.h   WSSAnvil.h (Size: 13.33 KB / Downloads: 237)
Reply
Thanks given by:




Users browsing this thread: 6 Guest(s)