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
Drop Python 3.6 support #4460
Merged
Merged
Drop Python 3.6 support #4460
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
3248518
Remove python 3.6 code
mariosasko 6ad94e1
Update requirements
mariosasko 3800c49
Style
mariosasko 32aab3d
Update audio gh action
mariosasko 2dee899
Benchmarks fix attempt #1
mariosasko 8c6ce19
Benchmarks fix attempt no.2
mariosasko 89b9ea6
Use newer image
mariosasko f9b24c4
Remove backticks
mariosasko 8a82d72
Add suggested command to benchmark action
mariosasko 3dbe753
Merge branch 'master' of github.com:huggingface/datasets into drop-py…
mariosasko 6eab0e4
Avoid some FutureWarnings and DeprecationWarnings
mariosasko 97ee698
Disable test
mariosasko 1cdcaa5
Merge conflicts
mariosasko 30de7fa
Remove 3.6 pickling test
mariosasko 983b04e
CI test
mariosasko eac1aaa
Use python 3.7 in ubuntu-latest
mariosasko 4db3cf9
Disable s3 test on Linux
mariosasko c4b4cb6
Merge branch 'main' of github.com:huggingface/datasets into drop-pyth…
mariosasko 2674f0c
Remove weird json file
mariosasko c3bc52d
Remove cloudpickle stuff
mariosasko ad949dd
Use lower torchaudio version
mariosasko feebd90
Try to fix s3 errors
mariosasko 59a1e3d
Another attempt
mariosasko 9c2b16c
Disable test
mariosasko File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think you need this ? The test currently passes on
main
on linuxThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It doesn't on Python 3.7. Notice the
test_dummy_dataset_serialize_s3
failure in one of the previous testThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok. This test is almost never run then, Maybe let's remove it completely in another PR, and make another one to make sure S3 export works as expected. Maybe we can use the
mockfs
fixture defined in #4724 insteadThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This issue seems related to getmoto/moto#4750, but neither bumping
moto
to3.0.0
(CI runs forever) nor pinning responses to0.16
(the serialization fails) helps, so I agree we can disable these tests for now and replace them in another PR with themockfs
fixture