8000 Fix missing material_to_h3 refs by BielStela · Pull Request #972 · Vizzuality/landgriffon · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix missing material_to_h3 refs #972

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 yo 8000 u account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 7, 2023
Merged

Conversation

BielStela
Copy link
Contributor
@BielStela BielStela commented Jun 6, 2023

I was reusing a cursor used as iterator in the loop. Worst practice but this behavior should be documented in https://www.psycopg.org/psycopg3/docs/api/cursors.html

…his fixes the issue where child materials that do not have an explicit raster source where not being linked in the material_to_h3 table
@vercel
Copy link
vercel bot commented Jun 6, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Ignored Deployments
Name Status Preview Updated (UTC)
landgriffon-client ⬜️ Ignored (Inspect) Jun 6, 2023 4:50pm
landgriffon-cookie-traceability ⬜️ Ignored (Inspect) Jun 6, 2023 4:50pm

@BielStela BielStela requested review from alexeh and elpamart June 6, 2023 16:50
@alexeh alexeh merged commit 42b4c3b into dev Jun 7, 2023
@alexeh alexeh deleted the data/fix/materal_to_h3_missing_links branch June 7, 2023 05:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0