Skip to content

Main#305

Merged
bputzeys merged 4 commits intoreleasefrom
main
Nov 20, 2025
Merged

Main#305
bputzeys merged 4 commits intoreleasefrom
main

Conversation

@maxiallard
Copy link
Contributor

This pull request introduces improvements to logging, expands support for sparse matrices in the scgpt dataset, and adds a new test for quantization initialization. The most impactful changes are the enhanced handling of sparse matrices for efficient data access and serialization, and the refinement of logging configuration for better clarity and control. Additionally, the package version is bumped and a dependency is added.

Logging improvements:

  • Refined logging setup in helical/__init__.py to use a dedicated helical logger, improved formatting to include timestamps, and disabled propagation to the root logger for clearer, project-specific log output.
  • Updated logging usage in helical/models/transcriptformer/utils/utils.py to use a module-specific logger instead of global configuration.

Sparse matrix support and dataset enhancements:

  • Added efficient handling of sparse matrices in scgpt dataset (helical/models/scgpt/dataset.py): tracks matrix type, handles serialization/deserialization, and retrieves rows as dense arrays only when needed, improving memory usage and performance. [1] [2]
  • Adjusted process_data in scgpt/model.py to use the raw adata.X matrix, supporting both sparse and dense formats.

Testing improvements:

  • Added a test for model initialization with quantization in test_c2s.py, verifying correct device assignment and quantization configuration.

Package management:

  • Bumped package version to 1.4.15 and added protobuf>=3.20.0 to dependencies in pyproject.toml. [1] [2]

raschedh and others added 4 commits November 19, 2025 09:02
* missing import and test for quantisation

* fix double logging

* add time and do not remove root loggers - not needed

---------

Co-authored-by: Maxime Allard <maxime@helical-ai.com>
Co-authored-by: Benoit Putzeys <157973952+bputzeys@users.noreply.github.com>
@bputzeys bputzeys merged commit 4ea3015 into release Nov 20, 2025
10 checks passed
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.

3 participants