Skip to content

feat: presets#3653

Open
flying-sheep wants to merge 69 commits intomainfrom
pa/presets
Open

feat: presets#3653
flying-sheep wants to merge 69 commits intomainfrom
pa/presets

Conversation

@flying-sheep
Copy link
Member

@flying-sheep flying-sheep commented May 28, 2025

  • Release notes not necessary because:

Self-documenting presets work!

E.g.:

@codecov
Copy link

codecov bot commented May 28, 2025

Codecov Report

❌ Patch coverage is 93.26923% with 14 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.34%. Comparing base (b5efe54) to head (ab0e916).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/scanpy/_settings/presets.py 91.08% 9 Missing ⚠️
src/scanpy/preprocessing/_scale.py 66.66% 2 Missing ⚠️
src/scanpy/tools/_leiden.py 75.00% 2 Missing ⚠️
src/scanpy/readwrite.py 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3653      +/-   ##
==========================================
+ Coverage   78.23%   78.34%   +0.11%     
==========================================
  Files         117      117              
  Lines       12661    12782     +121     
==========================================
+ Hits         9905    10014     +109     
- Misses       2756     2768      +12     
Flag Coverage Δ
hatch-test.low-vers 77.64% <93.26%> (+0.11%) ⬆️
hatch-test.pre 77.30% <93.26%> (+0.12%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
src/scanpy/__init__.py 100.00% <100.00%> (ø)
src/scanpy/_settings/__init__.py 86.54% <100.00%> (+0.50%) ⬆️
src/scanpy/_utils/__init__.py 75.28% <ø> (-0.11%) ⬇️
src/scanpy/experimental/pp/_normalization.py 94.25% <100.00%> (+0.06%) ⬆️
src/scanpy/plotting/_anndata.py 85.53% <100.00%> (ø)
src/scanpy/plotting/_baseplot_class.py 88.32% <100.00%> (ø)
src/scanpy/plotting/_dotplot.py 94.87% <100.00%> (ø)
src/scanpy/plotting/_matrixplot.py 96.51% <100.00%> (ø)
src/scanpy/plotting/_stacked_violin.py 85.85% <100.00%> (ø)
src/scanpy/plotting/_tools/__init__.py 76.17% <100.00%> (ø)
... and 15 more

@flying-sheep flying-sheep added this to the 1.12.0 milestone Jun 3, 2025
@flying-sheep flying-sheep changed the base branch from main to pa/settings-overhaul June 6, 2025 10:07
@flying-sheep flying-sheep modified the milestones: 1.12.0, 1.13.0 Nov 3, 2025
@flying-sheep
Copy link
Member Author

flying-sheep commented Nov 10, 2025

TODO: add jaccard to seurat preset #3828

@flying-sheep flying-sheep changed the title (feat): presets feat: presets Feb 23, 2026
@flying-sheep flying-sheep marked this pull request as ready for review March 23, 2026 16:43
@flying-sheep flying-sheep requested a review from ilan-gold March 23, 2026 16:47
@flying-sheep
Copy link
Member Author

flying-sheep commented Mar 23, 2026

@ilan-gold I asked your review because of scope:

Copy link
Contributor

@ilan-gold ilan-gold left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a pretty harmless PR so nothing comes to mind as absolutely pressing. The architecture overall makes sense.

I would beef up the docs a bit to make it clear how to use this.

Why use ScanpyV2Preview instead of just ScanpyV2?

@flying-sheep
Copy link
Member Author

Why use ScanpyV2Preview instead of just ScanpyV2?

Because when we release a new feature release before 2.0, I don’t want people to start relying on it keeping its behavior. I thought “preview” makes it clear that it’s not stable, but I’m open to suggestions.

Copy link
Contributor

@ilan-gold ilan-gold left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Modulo the repr (whcih I do think is worth discussion at the minmum if not a tweak), looks great!!!

@flying-sheep flying-sheep requested a review from ilan-gold March 26, 2026 12:55
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.

Scanpy 2.0 preview Remove default from sc.pp.scale’s zero_center (so it never accidentally densifies)

2 participants