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

Allow file-like objects when saving and loading #15

Merged
merged 3 commits into from
Feb 16, 2022

Conversation

agrinh
Copy link
Contributor

@agrinh agrinh commented Aug 19, 2021

Allow users to save the configs to arbitrary locations through file-like objects. Would e.g. simplify coqui-ai/TTS#683 without adding an fsspec dependency to this library.

@erogol
Copy link
Member

erogol commented Aug 25, 2021

@agrinh Sorry for the delay. I missed the PR among a ton of Github notifications.

I guess you reverted the changes you made in the first commit?

@agrinh
Copy link
Contributor Author

agrinh commented Aug 27, 2021

@erogol No worries 🙂 Sorry I'm confused, what changes do you mean I reverted? I don't think I've reverted anything.

Seems like the check was unhappy about some lint, so I rebased against master. Hope that this works.

@erogol erogol merged commit dabaf67 into coqui-ai:main Feb 16, 2022
@erogol
Copy link
Member

erogol commented Feb 16, 2022

late merge but good merge :)

@erogol
Copy link
Member

erogol commented Feb 16, 2022

I needed to revert it as it fails with p3.6

@agrinh
Copy link
Contributor Author

agrinh commented Feb 16, 2022

@erogol Is there a traceback somewhere?

@erogol
Copy link
Member

erogol commented Mar 2, 2022

@agrinh alas no

but contextlib.nullcontext is not available for p3.6

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