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

Include link to expanded grid #318

Merged
merged 2 commits into from
Oct 1, 2024
Merged

Include link to expanded grid #318

merged 2 commits into from
Oct 1, 2024

Conversation

JM-GA
Copy link
Contributor

@JM-GA JM-GA commented Sep 30, 2024

Include link to expanded grid. New grid has been uploaded to production

Copy link

github-actions bot commented Sep 30, 2024

Preview is ready!

View the preview: 🌐 PR Preview 318

You can also view the deploy log or read the documentation.

@benji-glitsos-ga
Copy link
Collaborator

@JM-GA I just made a minor edit

@robbibt
Copy link
Member

robbibt commented Sep 30, 2024

@JM-GA
Copy link
Contributor Author

JM-GA commented Sep 30, 2024

Hi @robbibt, I created a mapping between old and new grids, but then I realised someone had already created one that was stored in dev. I have removed the one I created, and copied the one from dev into prod

@JM-GA JM-GA merged commit 287ec99 into main Oct 1, 2024
1 check passed
@JM-GA JM-GA deleted the expanded-grid branch October 1, 2024 00:29
@robbibt
Copy link
Member

robbibt commented Oct 1, 2024

Hey @JM-GA, this is likely my fault as I think I originally generated this grid, but just noticed that we have some weird issues down in the south:
image

If your version of the grid doesn't have this problem, perhaps we could upload it and rename it "ga_summary_grid_c3_expanded.geojson"?

@JM-GA
Copy link
Contributor Author

JM-GA commented Oct 1, 2024

Thanks for checking @robbibt, I have swapped it out with my version - it seems to be ok

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.

3 participants