Skip to content

Conversation

@lguerard
Copy link
Contributor

@lguerard lguerard commented May 6, 2025

Fixes #95

@codecov
Copy link

codecov bot commented May 6, 2025

Codecov Report

❌ Patch coverage is 16.66667% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 25%. Comparing base (745fc61) to head (6efda65).
⚠️ Report is 22 commits behind head on devel.

Files with missing lines Patch % Lines
src/imcflibs/imagej/misc.py 17% 5 Missing ⚠️
Additional details and impacted files
@@         Coverage Diff          @@
##           devel   #101   +/-   ##
====================================
- Coverage     25%    25%   -0%     
====================================
  Files         25     25           
  Lines       1688   1696    +8     
====================================
+ Hits         421    422    +1     
- Misses      1267   1274    +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

@ehrenfeu ehrenfeu added this to the 2.0.0 milestone Jan 14, 2026
@ehrenfeu ehrenfeu added enhancement New feature or request new-minor Change introduced in PR will require a new minor release. labels Jan 14, 2026
@ehrenfeu ehrenfeu linked an issue Jan 14, 2026 that may be closed by this pull request
@ehrenfeu ehrenfeu moved this to In review in imcflibs Jan 14, 2026
@ehrenfeu ehrenfeu self-requested a review January 14, 2026 12:51
Copy link
Member

@ehrenfeu ehrenfeu left a comment

Choose a reason for hiding this comment

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

The function name convert_bytes isn't really telling what it's actually doing, it only reveals the bytes are converted to something unknown.

Please rename it to human_bytes or similar.

@ehrenfeu ehrenfeu moved this from In review to In progress in imcflibs Jan 14, 2026
@ehrenfeu ehrenfeu removed this from the 2.0.0 milestone Jan 14, 2026
@ehrenfeu
Copy link
Member

Removed it from the current milestone as this is probably not blocking other code - feel free to re-add if you need it already now.

@ehrenfeu ehrenfeu added the changelog Needs to be mentioned in release changelogs label Jan 14, 2026
@ehrenfeu
Copy link
Member

#126

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog Needs to be mentioned in release changelogs enhancement New feature or request new-minor Change introduced in PR will require a new minor release.

Projects

Status: In progress

Development

Successfully merging this pull request may close these issues.

Add method to convert bytes

2 participants