Skip to content

Enhance MapAggregator #70

New issue

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

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

Already on GitHub? # to your account

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

JaskRendix
Copy link
Contributor

@JaskRendix JaskRendix commented Mar 1, 2025

Waiting #62

PR improves the MapAggregator class:

  • Layering maps: you can now give each map a "z-index", so you can control which maps appear on top of others
  • Removing maps: you can now take maps out of the aggregator if you don't need them anymore
  • Tile size check: the code now makes sure all the maps you add have the same tile size, to prevent weird display issues
  • Fixes: some minor fixes and improvements to how the maps are positioned and sized
  • Unittest suite MapAggregator and the other two classes in data.py

@JaskRendix JaskRendix force-pushed the data-map branch 2 times, most recently from e2e4481 to 7cfed5c Compare March 23, 2025 12:32
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants