Skip to content

Conversation

@lyricnz
Copy link
Contributor

@lyricnz lyricnz commented Jul 11, 2023

Merging results as mentioned in #154

@lyricnz lyricnz changed the title Feature/update results interface Add extents to suburbs Jul 11, 2023
@lyricnz
Copy link
Contributor Author

lyricnz commented Jul 11, 2023

Do not merge this as-is. Requires a stop of the auto process, a merge and rerun locally, then a final commit.

@lyricnz lyricnz marked this pull request as draft July 11, 2023 23:03
@lyricnz
Copy link
Contributor Author

lyricnz commented Jul 17, 2023

I'll make this an auto-detect-and-run so we don't have to stop processing.

@lyricnz lyricnz force-pushed the feature/update-results-interface branch from c98982d to 37b88b9 Compare July 17, 2023 23:10
@lyricnz lyricnz marked this pull request as ready for review July 17, 2023 23:10
@lyricnz
Copy link
Contributor Author

lyricnz commented Jul 17, 2023

Ready to merge. Will add extents to the combined JSON file on first run. FWIW this increases file from 2.2M to 3.5M.

@LukePrior
Copy link
Owner

This looks good but I'm going to wait until work on the website is done so it doesn't unnecessarily increase the file size of results

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