Skip to content
This repository has been archived by the owner on Aug 2, 2023. It is now read-only.

fix: automatic round-down to quantum size #554

Merged
merged 17 commits into from
Mar 24, 2022

Conversation

kyujin-cho
Copy link
Member

@codecov
Copy link

codecov bot commented Mar 18, 2022

Codecov Report

Merging #554 (7e980ae) into main (2fc081d) will increase coverage by 0.00%.
The diff coverage is 30.55%.

@@           Coverage Diff           @@
##             main     #554   +/-   ##
=======================================
  Coverage   49.33%   49.33%           
=======================================
  Files          55       55           
  Lines        8973     8991   +18     
=======================================
+ Hits         4427     4436    +9     
- Misses       4546     4555    +9     
Impacted Files Coverage Δ
src/ai/backend/manager/models/image.py 49.13% <ø> (-0.26%) ⬇️
src/ai/backend/manager/scheduler/dispatcher.py 24.73% <0.00%> (-0.07%) ⬇️
src/ai/backend/manager/registry.py 18.29% <42.30%> (+0.72%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2fc081d...7e980ae. Read the comment docs.

@kyujin-cho kyujin-cho self-assigned this Mar 18, 2022
@kyujin-cho kyujin-cho added the bug label Mar 18, 2022
@kyujin-cho kyujin-cho added this to the 22.03 milestone Mar 18, 2022
src/ai/backend/manager/registry.py Outdated Show resolved Hide resolved
src/ai/backend/manager/registry.py Outdated Show resolved Hide resolved
src/ai/backend/manager/registry.py Outdated Show resolved Hide resolved
src/ai/backend/manager/registry.py Outdated Show resolved Hide resolved
src/ai/backend/manager/registry.py Outdated Show resolved Hide resolved
@kyujin-cho kyujin-cho requested a review from achimnol March 22, 2022 03:56
@achimnol achimnol merged commit b85d148 into main Mar 24, 2022
@achimnol achimnol deleted the fix/fractional-alloc-round-down branch March 24, 2022 15:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants