Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(scheduling): raise an error for invalid resources #2894

Merged
merged 1 commit into from
Aug 13, 2022

Conversation

bojiang
Copy link
Member

@bojiang bojiang commented Aug 11, 2022

No description provided.

@bojiang bojiang requested review from ssheng, parano and a team as code owners August 11, 2022 17:08
@bojiang bojiang requested review from aarnphm and removed request for a team August 11, 2022 17:08
@bojiang bojiang added the pr/merge-hold Requires further discussions before a pull request can be merged label Aug 11, 2022
@bojiang bojiang removed the pr/merge-hold Requires further discussions before a pull request can be merged label Aug 11, 2022
@bojiang bojiang mentioned this pull request Aug 11, 2022
5 tasks
@codecov
Copy link

codecov bot commented Aug 11, 2022

Codecov Report

Merging #2894 (0ce88b2) into main (7eab6ce) will decrease coverage by 0.69%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2894      +/-   ##
==========================================
- Coverage   70.85%   70.15%   -0.70%     
==========================================
  Files         103      120      +17     
  Lines        9328     9785     +457     
==========================================
+ Hits         6609     6865     +256     
- Misses       2719     2920     +201     
Impacted Files Coverage Δ
bentoml/_internal/runner/strategy.py 84.21% <0.00%> (+10.07%) ⬆️
bentoml/xgboost.py 0.00% <0.00%> (ø)
bentoml/onnx.py 0.00% <0.00%> (ø)
bentoml/transformers.py 0.00% <0.00%> (ø)
bentoml/mlflow.py 0.00% <0.00%> (ø)
bentoml/exceptions.py 93.33% <0.00%> (ø)
bentoml/keras.py 0.00% <0.00%> (ø)
bentoml/picklable_model.py 0.00% <0.00%> (ø)
bentoml/torchscript.py 55.55% <0.00%> (ø)
bentoml/sklearn.py 0.00% <0.00%> (ø)
... and 14 more

@ssheng
Copy link
Collaborator

ssheng commented Aug 12, 2022

There were CI test failures. Could you please take a look, @bojiang?

@aarnphm aarnphm changed the title fix(scheduling): raise an error if user provide a invalid resources request fix(scheduling): raise an error for invalid resources Aug 13, 2022
@aarnphm aarnphm merged commit b63b69a into bentoml:main Aug 13, 2022
aarnphm pushed a commit that referenced this pull request Aug 14, 2022
Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
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.

None yet

3 participants