Fractional CUs#54

Merged

denisbsu merged 15 commits intomasterfrom

May 4, 2026

Contributor

denisbsu commented Apr 30, 2026

No description provided.

denisbsu added 5 commits

April 29, 2026 11:44


          fractional CUs for small queries

bf5e4e9


          fractional claim

11cb44c


          chip calculation fix

260e2b1


          Force backoff on overload

8deaf83


          Remove unnecessary comments

22cce8b

denisbsu requested a review from kalabukdima

April 30, 2026 14:48


          Correct Error Type and cargo fmt

35c0b72

kalabukdima reviewed

View reviewed changes

src/controller/p2p.rs


          Add failing test proving fractional CU refund bug in Bucket::put

28bf25e

put() has min/max reversed: allocation_chip.min(0.).max(1.) always
returns 1.0, so every fractional refund credits a full token.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

mo4islona reviewed

View reviewed changes

src/compute_units/rate_limiter.rs Outdated

mo4islona reviewed

View reviewed changes

src/controller/p2p.rs Outdated

mo4islona reviewed

View reviewed changes

src/controller/p2p.rs Outdated

mo4islona reviewed

View reviewed changes

src/compute_units/rate_limiter.rs Outdated

mo4islona reviewed

View reviewed changes

src/controller/p2p.rs

mo4islona requested changes

View reviewed changes

denisbsu added 6 commits

May 4, 2026 12:34


          Incorrect refund fixed

9efecda


          Fix underflow

b9f0d63


          More appropriate name for is_empty

e84abab


          comments on how RL expected to work

b6cb3ae


          comments on errors

7db66f9


          cargo fmt

49f69ec

kalabukdima approved these changes

View reviewed changes

src/controller/p2p.rs Outdated

src/controller/p2p.rs Outdated

src/compute_units/rate_limiter.rs Outdated

denisbsu added 2 commits

May 4, 2026 18:54


          Typos

c373124


          max.min -> clamp

denisbsu merged commit 70dbfe0 into master

3 checks passed

denisbsu deleted the fractional_cu branch

May 4, 2026 16:01

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

Labels

None yet