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: make sure not to override user set values for from_sample #3610

Merged
merged 3 commits into from
Mar 2, 2023

Conversation

aarnphm
Copy link
Contributor

@aarnphm aarnphm commented Feb 23, 2023

Signed-off-by: Aaron [email protected]

@aarnphm aarnphm requested a review from a team as a code owner February 23, 2023 06:17
@aarnphm aarnphm requested review from parano and removed request for a team February 23, 2023 06:17
@codecov
Copy link

codecov bot commented Feb 23, 2023

Codecov Report

Merging #3610 (cd61db6) into main (e5af0c3) will decrease coverage by 0.02%.
The diff coverage is 50.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3610      +/-   ##
==========================================
- Coverage   30.46%   30.44%   -0.02%     
==========================================
  Files         143      143              
  Lines       11834    11840       +6     
  Branches     1950     1956       +6     
==========================================
  Hits         3605     3605              
  Misses       7985     7985              
- Partials      244      250       +6     
Impacted Files Coverage Δ
src/bentoml/_internal/io_descriptors/numpy.py 41.66% <50.00%> (-0.39%) ⬇️
src/bentoml/_internal/io_descriptors/pandas.py 39.01% <50.00%> (-0.44%) ⬇️

sauyon
sauyon previously approved these changes Feb 23, 2023
Signed-off-by: Aaron <[email protected]>
@aarnphm aarnphm merged commit 246ad6c into bentoml:main Mar 2, 2023
@aarnphm aarnphm deleted the fix/sample-dtypes branch March 2, 2023 04:00
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.

2 participants