search

aws / sagemaker-huggingface-inference-toolkit

Apache License 2.0
240 stars 60 forks source link

Fix input data used in test_decode_csv #38

Closed dlesomewhere closed 2 years ago

dlesomewhere commented 2 years ago

Issue #38

test_decode_csv fails when I run the tests locally. Looking at the implementation of the test I think there's an error in the input data for the second assertion.

Description of changes:

This commit fixes the input data for the second assertion so that the test passes.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

philschmid commented 2 years ago

Hey @dlesomewhere,

Thank you for finding the issue and opening the PR. I finally added CI-Pipeline for code quality and tests in https://github.com/aws/sagemaker-huggingface-inference-toolkit/pull/42, which forced me to fix it directly.