Commit Graph
5 Commits
Author SHA1 Message Date
Cliff Meyers 61903939a3 remove __lt__ from RegionSet & wrapper as it's no longer used in genPOI 2020-04-15 08:06:12 -04:00
Cliff Meyers 512d6e7583 use logging.debug for detailed reporting of entities found during genPOI 2020-04-15 07:52:34 -04:00
Cliff Meyers 0a1560431a fix a bug where multiple renders of the same world led to repeat genPOI scans
the code that calls handleEntities in genPOI was building a list
of all filter funcs in the config, then used itertools.groupby to walk over
each region set, calling handleEntities and passing filters funcs for the rset.

this works fine when all of the renders point at a different world,
since rsets are sorted via __lt__ using the regiondir. when an rset appears in
the config multiple times, groupby can't sort the rsets reliably,
so a single rset is passed multiple times to handleEntities with only some
of the applicable filters. for a config with just two renders of the same world,
handleEntities would frequently be called 5-7 times instead of just 2.

this change eliminates group by, and just iterates over all the rsets, each
time plucking the list of applicable filters based on rset associated with
each filter (simple equality).
2020-04-15 07:34:54 -04:00
Cliff Meyers 80e45c4d66 genPOI: apply consistent sorting to markers.js and markersDB.js
use json.dumps(sort_keys=True) to apply a consistent sorting to the output JS files.
improves the readability of diffs and making config testing easier
2020-04-05 08:29:17 -04:00
Cliff Meyers a29e9ddf20 Track centers by layer instead of by world
This fixes a bug where a world with multiple layers with different
"northdirection" values would initially load at the wrong center point
because the center for the last layer would override prior values
in the overview.collections.centers map.
2019-12-30 19:40:36 -05:00