Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tiled #67

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Tiled #67

wants to merge 11 commits into from

Conversation

wziard
Copy link

@wziard wziard commented Nov 17, 2018

A new pull request for just the code for the tiled output.

I thought you might maybe be interested in this part, because it is also a fix for issue #48.

It still needs some work to fix the discontinuities in the height shading on the tile edges, but I'll have to look into that more before I try to fix that.

@wziard
Copy link
Author

wziard commented Nov 17, 2018

Accidentally committed a type, force pushed the correct version just now.

@wziard
Copy link
Author

wziard commented Nov 26, 2018

I renamed a function and I think the code is quite OK now. Might need some more commenting. I'll try to do that soon(ish)

Martijn Versteegh and others added 4 commits November 27, 2018 10:41
The produces tiled output way faster than repeatedly using --geometry
because it only reads the databse once.
Negative x/z will be put into negative tiles, so you can easily determine
the tile location by dividing the x or z by 16*tileSize. This means the same area of the world
will always end up in the same tile, independent of how much of the world is generated.
Martijn Versteegh and others added 4 commits November 28, 2018 15:42
Apparently ofstream.open() only takes a char const * and not a std::string in Clang++;
@wziard
Copy link
Author

wziard commented Nov 30, 2018

I refactored the code a bit and renamed some unclear variable/function names. Also laid the groundwork for building the zoom pyramid in a separate utility. I think it's ready to be pulled now :-)

…with, prepend "tile_" to the tile filenames.
@Opvolger
Copy link

Opvolger commented Nov 7, 2020

./minetestmapper -i /opt/docker/minetest/data/worlds/world/ -o ~/pxe/volume_pxe/map.png --zoom 4 --drawplayers --tilesize 4096x4096
[sudo] password for opvolger:
Warning: Falling back to using colors.txt from current directory.
Warning: The width or height of the image to be created exceeds 4096 pixels! (Dimensions: 16384x16384)
Warning: could not write to 'metadata_/home/opvolger/pxe/volume_pxe/map.png.txt'!

metadata_ dir is not correct.

@Opvolger
Copy link

Opvolger commented Nov 7, 2020

wrote image:-1_-1_map.png
wrote image:-1_0_map.png
wrote image:-1_1_map.png
wrote image:-1_2_map.png
wrote image:-1_3_map.png

negative ?

@wziard
Copy link
Author

wziard commented Nov 8, 2020

negative ?

Yes. Why not? 0_0_map.png is the center of the world.

This way the tile names are always the same, no matter what part of the world you request.

@sfan5 sfan5 removed their request for review August 23, 2022 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants