Conversation
Codecov Report❌ Patch coverage is
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
Flags with carried forward coverage won't be shown. Click here to find out more.
|
This reverts commit b158545.
|
TODO: add |
|
@ilan-gold I asked your review because of scope:
|
ilan-gold
left a comment
There was a problem hiding this comment.
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?
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. |
ilan-gold
left a comment
There was a problem hiding this comment.
Modulo the repr (whcih I do think is worth discussion at the minmum if not a tweak), looks great!!!
Self-documenting presets work!
E.g.:
scanpy/src/scanpy/_settings.py
Lines 90 to 96 in 3bae7ae