Skip to content

(fix): add dask upper bound for sparse issue#1862

Merged
ilan-gold merged 5 commits into
mainfrom
ig/dask_bound
Feb 18, 2025
Merged

(fix): add dask upper bound for sparse issue#1862
ilan-gold merged 5 commits into
mainfrom
ig/dask_bound

Conversation

@ilan-gold
Copy link
Copy Markdown
Contributor

@ilan-gold ilan-gold commented Feb 17, 2025

  • Closes #
  • Tests added
  • Release note added (or unnecessary)

dask/dask#11752

@ilan-gold ilan-gold added this to the 0.11.4 milestone Feb 17, 2025
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 17, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.84%. Comparing base (9838253) to head (7f87e9c).
Report is 65 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1862      +/-   ##
==========================================
- Coverage   86.25%   83.84%   -2.42%     
==========================================
  Files          41       41              
  Lines        6288     6288              
==========================================
- Hits         5424     5272     -152     
- Misses        864     1016     +152     

see 8 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Member

@flying-sheep flying-sheep left a comment

Choose a reason for hiding this comment

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

what’s the logic for the changed bounds here?

@ilan-gold
Copy link
Copy Markdown
Contributor Author

what’s the logic for the changed bounds here?

The low bound was copy-paste miskate, the upper bound is the added issue

@flying-sheep flying-sheep changed the title (fix): add dask bound for sparse issue (fix): add dask upper bound for sparse issue Feb 18, 2025
@ilan-gold ilan-gold merged commit 7fc66bb into main Feb 18, 2025
@ilan-gold ilan-gold deleted the ig/dask_bound branch February 18, 2025 10:50
meeseeksmachine pushed a commit to meeseeksmachine/anndata that referenced this pull request Feb 18, 2025
ilan-gold added a commit that referenced this pull request Feb 18, 2025
Co-authored-by: Ilan Gold <ilanbassgold@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants