nbn-upgrade-map icon indicating copy to clipboard operation
nbn-upgrade-map copied to clipboard

Consider minimizing the json files to reduce repo size?

Open lyricnz opened this issue 2 years ago • 3 comments

If we're not going to gzip the geojson files per https://github.com/LukePrior/nbn-upgrade-map/issues/122 maybe we should minimize the file-size (though this would make native diffs useless, without reformatting both the before and after).

File-size is about 25% smaller when storing JSON without additional whitespace

json.dumps(d, separators=(',', ':'))

lyricnz avatar Jul 09 '23 02:07 lyricnz

That's probably a decent idea for the immediate turn as that won't break any functionality.

I'm not ruling out zipping everything in future but that will require more work on front and backend.

Once we start reprocessing suburbs we should look into generating statistics about the changes

LukePrior avatar Jul 09 '23 09:07 LukePrior

@LukePrior Do we want to minimise the old ones (big diff, big change history), or just change it moving forwards?

lyricnz avatar Jul 11 '23 00:07 lyricnz

Updated PR

lyricnz avatar Aug 02 '23 05:08 lyricnz

Closing as "not required"

lyricnz avatar Jul 08 '24 08:07 lyricnz