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.

Sign up for GitHub

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

Jump to bottom

Remove pandas #3

Merged
merged 3 commits into from
Mar 6, 2024
Merged

Remove pandas #3

merged 3 commits into from
Mar 6, 2024

Conversation

eginhard
Copy link
Owner

@eginhard eginhard commented Mar 6, 2024

Copied from coqui-ai#3133:

pandas was only used in a few places to read or write CSV files, which can just as well be done with the standard library. I've changed that accordingly and moved pandas to the optional notebooks requirements.

I also removed the separate numba pin for Python < 3.9 because only Python >= 3.9 is supported now.

@eginhard eginhard merged commit f24f7c1 into dev Mar 6, 2024
33 of 34 checks passed
@eginhard eginhard deleted the remove-pandas branch March 6, 2024 21:38
@eginhard
Copy link
Owner Author

eginhard commented Mar 7, 2024

Fixed coqui-ai#3330

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@eginhard