Skip to content

⚡ Use Rust to Save JSON files - #1127

Open
hannah275 wants to merge 112 commits into
TissueImageAnalytics:developfrom
hannah275:rust-json
Open

⚡ Use Rust to Save JSON files#1127
hannah275 wants to merge 112 commits into
TissueImageAnalytics:developfrom
hannah275:rust-json

Conversation

@hannah275

@hannah275 hannah275 commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator

This work focuses on converting code in tiatoolbox/utils/misc.py and tiatoolbox/models/engine/multi_task_segmentor.py from Python to Rust with the aim of reducing the time it takes to run the code. This Jupyter notebook shows the speedup that certain functions are having when written in Rust.

Issues:
-If the number of bytes needed goes above 32 an error would occur
-Currently test_rust.py is running during the Python and Rust tests

@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@hannah275 hannah275 changed the title Rust json Use rust to save JSON files Aug 24, 2026
@shaneahmed shaneahmed changed the title Use rust to save JSON files ⚡ Use Rust to Save JSON files Aug 24, 2026
@codecov

codecov Bot commented Aug 24, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 85.53299% with 57 lines in your changes missing coverage. Please review.
✅ Project coverage is 99.45%. Comparing base (320ef7a) to head (80ac12e).
⚠️ Report is 2 commits behind head on develop.

Files with missing lines Patch % Lines
tiatoolbox/rust-library/multitask/src/lib.rs 12.30% 57 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1127      +/-   ##
===========================================
- Coverage    99.90%   99.45%   -0.46%     
===========================================
  Files           93       96       +3     
  Lines        11920    12283     +363     
  Branches      1562     1557       -5     
===========================================
+ Hits         11909    12216     +307     
- Misses           5       62      +57     
+ Partials         6        5       -1     
Flag Coverage Δ
rust 84.46% <84.46%> (?)

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

☔ View full report in Codecov by Harness.
📢 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.

@shaneahmed
shaneahmed requested review from abdol and shaneahmed August 28, 2026 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants