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

Support dumping/rendering a selection of regions #10

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

samszotkowski
Copy link
Contributor

@samszotkowski samszotkowski commented Aug 21, 2024

This lets us specify in version.txt if we want to dump/render any particular regions (e.g. only ones that have changed since previous cache). stitch.py still needs access to all of the base tiles whether those particular regions are selected or not, so it can render plane 0 without any stitch lines, and for the zoomed out images, of which a selected region might only be a tiny part.

Note that it doesn't just render the selected regions, but also all adjacent regions in case map icons have been altered that would cross the boundary of the selected regions into adjacent ones.

@allenkinzalow
Copy link
Contributor

Screenshot 2024-08-23 at 10 14 28 PM

It looks like it is rendering the map icons in other regions, but not the other regions.

@samszotkowski
Copy link
Contributor Author

samszotkowski commented Aug 24, 2024

Interesting, did you have anything in your base tiles folder before running or did you start from scratch w/ a selection?

@samszotkowski
Copy link
Contributor Author

The idea of selections was that you have the previous tiles already rendered and you just need to add or replace some, so using a selection assumes the base tiles directory is already full. It will use the existing base tiles to generate those zoomed out images, but if they don't exist then you'll get garbage... I think that's what's happening here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants