Closed lyricnz closed 2 months ago
Should I remove the conversion to sqllite from the job? Nobody except me probably uses it anyway (the postgres DB image is too big for my puny machine).
I don't mind either way, if you find it useful feel free to leave it in.
It failed to run last time in the GHA anyway
I'm doing a local test of the new image here now, but my machine struggles to even rebuild the image.
Seemed to work fine (using local lukeprior/nbn-upgrade-map-db:202408 image)
❯ ./code/main.py --suburb Marian --state QLD
2024-09-08 13:35:54,607 INFO MainThread Checking for externally updated geojson results...
2024-09-08 13:36:01,535 INFO MainThread ...done
2024-09-08 13:36:01,645 INFO MainThread Creating DB index...
2024-09-08 13:36:01,682 INFO MainThread Selecting explicit Marian, QLD
2024-09-08 13:36:01,682 INFO MainThread Processing Marian, QLD
2024-09-08 13:36:01,682 INFO MainThread Fetching all addresses for Marian, QLD
2024-09-08 13:36:01,728 INFO MainThread Fetched 1819 addresses from database
2024-09-08 13:36:01,733 INFO MainThread Loaded 1631 addresses from output file
2024-09-08 13:36:01,734 INFO MainThread Submitting 1819 requests to add NBNco data...
...
2024-09-08 13:40:33,658 INFO nbn_3 Completed 1819 requests
2024-09-08 13:40:33,664 INFO MainThread Completed. Tally of tech types: {'WIRELESS': 418, 'FTTN': 872, 'FTTP': 331, None: 188, 'NULL': 6, 'SATELLITE': 4}
2024-09-08 13:40:33,665 INFO MainThread Location ID types: {'LOC': 1631, 'None': 188}
2024-09-08 13:40:33,666 INFO MainThread Writing results to results/QLD/marian.geojson
2024-09-08 13:40:34,077 INFO MainThread Updating progress.json
Diff looks reasonable.
Ok looks like that went ok
Getting some no valid suggestion warnings in the current automatic CI run but not sure if more than usual baseline.
Like #370