Conversation
Highlight key features added since v0.1.0: cross-model measures, multistage queries, dynamic joins with diamond support, model filters, DuckDB (Tier 1), query introspection, simpler query syntax, and SQL-style filters. Add Tutorials section linking to the docs/examples Jupyter notebooks. Promote DuckDB to CI-tested in the dialects list. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
📝 WalkthroughWalkthroughThe README was updated to document new capabilities introduced in version 0.1 and later, including advanced query features, multi-model support, and database compatibility. A new Tutorials section references Jupyter notebook examples organized by topic. The SQL dialect support count was increased from 13 to 14. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
README.md (1)
389-396: Consider linking directly to notebook files (or hosted docs pages).Line 389–396 currently links to folders. Switching to explicit
.ipynbfiles (or stable docs URLs) usually gives a more predictable click path across GitHub/PyPI/docs renderers.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@README.md` around lines 389 - 396, The README table currently links to directory paths (e.g., docs/examples/02_sql_vs_dsl/, docs/examples/03_auto_ingest/, etc.); update each link to point to the explicit notebook file or stable hosted docs URL (for example replace docs/examples/02_sql_vs_dsl/ with the exact .ipynb file or the canonical docs page for "SQL vs DSL") so clicks open the notebook/page directly; ensure each target exists in the repo or docs site and keep the visible link text (e.g., "SQL vs DSL", "Auto-Ingestion", "Time Operations", "Joins", "Joined Measures", "Multistage Queries") unchanged.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@README.md`:
- Around line 389-396: The README table currently links to directory paths
(e.g., docs/examples/02_sql_vs_dsl/, docs/examples/03_auto_ingest/, etc.);
update each link to point to the explicit notebook file or stable hosted docs
URL (for example replace docs/examples/02_sql_vs_dsl/ with the exact .ipynb file
or the canonical docs page for "SQL vs DSL") so clicks open the notebook/page
directly; ensure each target exists in the repo or docs site and keep the
visible link text (e.g., "SQL vs DSL", "Auto-Ingestion", "Time Operations",
"Joins", "Joined Measures", "Multistage Queries") unchanged.
Highlight key features added since v0.1.0: cross-model measures, multistage queries, dynamic joins with diamond support, model filters, DuckDB (Tier 1), query introspection, simpler query syntax, and SQL-style filters. Add Tutorials section linking to the docs/examples Jupyter notebooks. Promote DuckDB to CI-tested in the dialects list.
Summary by CodeRabbit