Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Add extents to suburbs #178

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

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