P14: Retrofit 5 notebooks to conf.connect factories #80

Merged
kert merged 1 commits from feature/notebook-retrofit into main 2026-03-22 00:31:54 +00:00
Owner

Replace hand-rolled connection boilerplate in 5 notebooks:

Notebook Before After
acodb_explorer 10-line sys.path + duckdb + nature connect.theme() + connect.duckdb()
pfs_calcs Manual duckdb.connect connect.duckdb()
bib_explorer Manual Store() connect.bib()
zotero_tutorial Manual sqlite3 connect.zotero()
cms_quality_measures Manual duckdb connect.duckdb()

All 5 notebooks run as scripts, all tests pass.

Closes #78

Replace hand-rolled connection boilerplate in 5 notebooks: | Notebook | Before | After | |----------|--------|-------| | acodb_explorer | 10-line sys.path + duckdb + nature | `connect.theme()` + `connect.duckdb()` | | pfs_calcs | Manual duckdb.connect | `connect.duckdb()` | | bib_explorer | Manual Store() | `connect.bib()` | | zotero_tutorial | Manual sqlite3 | `connect.zotero()` | | cms_quality_measures | Manual duckdb | `connect.duckdb()` | All 5 notebooks run as scripts, all tests pass. Closes #78
kert added this to the P14: Notebook DX — connections, snippets, templates, S3 buckets milestone 2026-03-22 00:31:33 +00:00
kert added 1 commit 2026-03-22 00:31:33 +00:00
retrofit 5 notebooks to use conf.connect factories
All checks were successful
ci/woodpecker/push/ci Pipeline was successful
ci/woodpecker/pr/ci Pipeline was successful
ci/woodpecker/pr/infra-ci Pipeline was successful
ci/woodpecker/push/infra-ci Pipeline was successful
coverage 99% coverage
15ccb08841
Replace hand-rolled connection boilerplate with conf.connect:
- acodb_explorer.py: connect.theme() + connect.duckdb()
- pfs_calcs.py: connect.duckdb()
- bib_explorer.py: connect.bib()
- zotero_tutorial.py: connect.zotero()
- cms_quality_measures.py: connect.duckdb()

Skipped: api_explorer (httpx test harness), nessie/polaris tutorials
(complex flows), gpu_test/sample (no DB), sql_generator (no DB).

fix #78
kert merged commit be8b0cfcc5 into main 2026-03-22 00:31:54 +00:00
Sign in to join this conversation.