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

Added example to README.me showing how to embed a single dictionary. #38

Merged
merged 1 commit into from
Jan 24, 2024
Merged

Conversation

WyvernIXTL
Copy link
Contributor

I hadn't thought too hard when reading the examples in README.md and in the documentation and just used the embed_all feature.
That's quite stupid though. A much better way is to embed the dictionary individually.

I just want to give people the idea to do so.

@tapeinosyne tapeinosyne merged commit 4e01786 into tapeinosyne:master Jan 24, 2024
1 check passed
@tapeinosyne
Copy link
Owner

Thanks.

Truth be told, the library as a whole is overdue an update, and that includes improvements to the way dictionaries are handled. It shouldn't be too long before the next release.

@WyvernIXTL
Copy link
Contributor Author

Good to hear. Thanks for merging 🙏

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