Shared datasets integration - #202
Draft
jonaraphael wants to merge 77 commits into
Draft
jonaraphael wants to merge 77 commits into
jonaraphael wants to merge 77 commits into
Conversation
GOOGLE_APPLICATION_CREDENTIALS not needed
# Conflicts: # .claude/skills/README.md # .claude/skills/cerulean-cloud-test-env/SKILL.md # .claude/skills/cerulean-cloud-test-env/agents/openai.yaml # .claude/skills/cerulean-ui-frontend-sql/SKILL.md # .claude/skills/cerulean-ui-frontend-sql/references/frontend-sql-inventory.md # .claude/skills/skytruth-intent-engineering/SKILL.md # AGENTS.md
# Conflicts: # stack/cloud_run_tipg.py
# Conflicts: # stack/cloud_run_tipg.py
…an-cloud into aoi-orchestrator
# Conflicts: # cerulean_cloud/database_schema.py
… can display multiples
The no-PostGIS fallback schema in conftest was missing the model table's computed zoom_level/scale columns. Also gitignore the repo-local .conda/ test env. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
f9e1c5e (May 3) -> c69850b (Aug 6, current main tip; SDK last changed at 64cc7f9). fetch_dataset/DatasetRef surface unchanged; full test suite passes locally against the new pin. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Same step the backfill branch runs. Locally the full suite passes under real PostGIS (79 passed / 0 failed), while the no-PostGIS compat fallback cannot run the slick_to_aoi tests (computed columns call ST_* functions), so real PostGIS in CI is required for a green, meaningful run on this branch. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
🍹
|
🍹
|
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Draft PR to run CI on
feat/shared-datasets.This branch integrates shared-datasets-1 as the source for AOI datasets (catalog-driven accessors, SDK-based fetches, user-AOI child-table geometry).
Latest commits worth reviewer attention:
feat/aoi_backfill). Locally the full suite passes under real PostGIS (79 passed / 0 failed); the no-PostGIS compat fallback cannot run the slick_to_aoi tests, sinceslick's computed columns callST_*functions.skytruth-shared-datasetspin bumpedf9e1c5e(May 3) →c69850b(Aug 6, current main tip).fetch_dataset/DatasetRefsurface unchanged; suite green locally against the new pin.modeltable gains thezoom_level/scalecolumns it was missing.🤖 Generated with Claude Code