Skip to content

feat: Extract make_streaming_source into _streaming module - #383

Open
WindowGenerator wants to merge 1 commit into
biodatageeks:masterfrom
WindowGenerator:refactor_streaming_source_callbacks
Open

feat: Extract make_streaming_source into _streaming module#383
WindowGenerator wants to merge 1 commit into
biodatageeks:masterfrom
WindowGenerator:refactor_streaming_source_callbacks

Conversation

@WindowGenerator

@WindowGenerator WindowGenerator commented May 10, 2026

Copy link
Copy Markdown

Introduce polars_bio/_streaming.py with make_streaming_source, StreamingConfig, PredicatePushdownConfig, pyarrow_schema_to_polars_dict, and extract_column_names. Refactor io.py, pileup_op.py, range_op_io.py, and utils.py to use the shared helpers, removing duplicated streaming logic and the tqdm dependency from each call-site.

Introduce polars_bio/_streaming.py with make_streaming_source,
StreamingConfig, PredicatePushdownConfig, pyarrow_schema_to_polars_dict,
and extract_column_names. Refactor io.py, pileup_op.py, range_op_io.py,
and utils.py to use the shared helpers, removing duplicated streaming
logic and the tqdm dependency from each call-site.
@WindowGenerator
WindowGenerator force-pushed the refactor_streaming_source_callbacks branch from d76588c to 047a858 Compare May 10, 2026 18:03
@WindowGenerator WindowGenerator changed the title Refactor streaming source callbacks feat: Extract make_streaming_source into _streaming module May 10, 2026
@WindowGenerator
WindowGenerator force-pushed the refactor_streaming_source_callbacks branch from 047a858 to 8369641 Compare May 10, 2026 18:07
@mwiewior

Copy link
Copy Markdown
Collaborator

hey @WindowGenerator - thanks a lot for these 3 PRs! All of them are relevant are on our short-term todo list. Please be patient as need to review and test them one by one carefully before accepting and merging. I will keep you posted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants