Skip to content

u64 for Fuel#308

Merged
florianhartung merged 1 commit intomainfrom
dev/u64-for-fuel
Jan 19, 2026
Merged

u64 for Fuel#308
florianhartung merged 1 commit intomainfrom
dev/u64-for-fuel

Conversation

@florianhartung
Copy link
Copy Markdown
Collaborator

@florianhartung florianhartung commented Jan 15, 2026

Pull Request Overview

TODO or Help Wanted

Checks

  • Using Nix
    • Ran nix fmt
    • Ran nix flake check '.?submodules=1'
  • Using Rust tooling
    • Ran cargo fmt
    • Ran cargo test
    • Ran cargo check
    • Ran cargo build
    • Ran cargo doc

Benchmark Results

Github Issue

Closes #307

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 15, 2026

Codecov Report

❌ Patch coverage is 69.69697% with 10 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/execution/interpreter_loop.rs 55.55% 8 Missing ⚠️
src/execution/checked/mod.rs 80.00% 1 Missing ⚠️
src/execution/config.rs 80.00% 1 Missing ⚠️
Files with missing lines Coverage Δ
src/execution/linker.rs 100.00% <100.00%> (ø)
src/execution/resumable.rs 100.00% <ø> (ø)
src/execution/store/mod.rs 86.16% <100.00%> (ø)
src/execution/checked/mod.rs 65.84% <80.00%> (ø)
src/execution/config.rs 81.25% <80.00%> (ø)
src/execution/interpreter_loop.rs 76.18% <55.55%> (+<0.01%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@florianhartung florianhartung force-pushed the dev/u64-for-fuel branch 3 times, most recently from eb6c05a to 47fb772 Compare January 19, 2026 13:39
@florianhartung florianhartung marked this pull request as ready for review January 19, 2026 13:39
@wucke13
Copy link
Copy Markdown
Collaborator

wucke13 commented Jan 19, 2026

I guess the reason for this is that u32 fuel is not lasting that long? with about 9 ns per fuel?

Signed-off-by: Florian Hartung <florian.hartung@dlr.de>
@florianhartung
Copy link
Copy Markdown
Collaborator Author

@wucke13 Yes, this is described in the related issue #307

@florianhartung florianhartung added this pull request to the merge queue Jan 19, 2026
Merged via the queue into main with commit 657fe0a Jan 19, 2026
12 of 13 checks passed
@florianhartung florianhartung deleted the dev/u64-for-fuel branch January 19, 2026 15:07
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.

Change fuel type from u32 to u64

2 participants