Skip to content

Fix/all neighbors heuristics - #822

Merged
Intron7 merged 3 commits into
mainfrom
fix/all_neighbors_heuristics
Oct 2, 2026
Merged

Intron7 merged 3 commits into
mainfrom
fix/all_neighbors_heuristics

Conversation

@Intron7

@Intron7 Intron7 commented Oct 1, 2026

Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: Intron7 <sdicks@nvidia.com>
Signed-off-by: Intron7 <sdicks@nvidia.com>
…ristics

Signed-off-by: Intron7 <sdicks@nvidia.com>
@codecov-commenter

codecov-commenter commented Oct 2, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 74.62687% with 17 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.13%. Comparing base (9040299) to head (57c37d3).

Files with missing lines Patch % Lines
...rocessing/_neighbors/_algorithms/_all_neighbors.py 65.95% 16 Missing ⚠️
src/rapids_singlecell/tools/_umap.py 94.44% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #822      +/-   ##
==========================================
- Coverage   90.22%   90.13%   -0.10%     
==========================================
  Files         123      123              
  Lines       12502    12559      +57     
==========================================
+ Hits        11280    11320      +40     
- Misses       1222     1239      +17     
Files with missing lines Coverage Δ
...ds_singlecell/preprocessing/_neighbors/__init__.py 90.29% <100.00%> (ø)
...ecell/preprocessing/_neighbors/_helper/__init__.py 84.72% <ø> (-0.42%) ⬇️
src/rapids_singlecell/tools/_umap.py 97.18% <94.44%> (-0.94%) ⬇️
...rocessing/_neighbors/_algorithms/_all_neighbors.py 81.08% <65.95%> (-11.78%) ⬇️

@Intron7
Intron7 merged commit 140a08b into main Oct 2, 2026
26 of 29 checks passed
@Intron7
Intron7 deleted the fix/all_neighbors_heuristics branch October 2, 2026 12:19
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