AllenInstitute / AllenInstitute/ConnectsCommonConnectivity

Register SynapseConnectivityLong with a bulk validation mode

Open
#19 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement io
Dominant language
Jupyter Notebook
Stars
4
Forks
1
PR merge metrics
No merged PRs in 30d

Description

Problem: registry entry is commented out because validate_for_write re-validates every row with pydantic — unusable at 8M rows; etl_v1dd_03 bypasses write_models with raw write_deltalake. A dead import remains in write_spec.py.

Fix: add a bulk mode (arrow-schema-level validation, optional row sampling); register overwrite_scoped on (project_id, dataset_id); route the notebook through write_models; remove the dead import.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the commented SynapseConnectivityLong registry entry and the validation path in write_spec.py, including validate_for_write and the dead import. Then inspect the etl_v1dd_03 notebook and its write_models/write_deltalake calls. Done means bulk Arrow-schema validation with optional sampling, scoped overwrite on project_id and dataset_id, notebook routing through write_models, and removal of the dead import.

Written by the indexing model from the issue text.

Assessment

Tech stack
jupyter-notebook, python
Domain
data-engineering, databases
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.