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

Adds a Blog Entry: How to get a Rust-based Package to CRAN #41

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

DavZim
Copy link

@DavZim DavZim commented Nov 7, 2024

Hi all,
as promised in the Discord Server: here is a first draft of a blog article.
What do you think?
For now there is at least an image (and alt-text) missing.

Copy link
Contributor

@JosiahParry JosiahParry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you so much for this draft! I hope you dont mind a little bit of editorial review!

I would love, love, love! if you did more exposition on the package itself and how it is useful. Particularly because people talk about how R can't be used for ML (wrong!) and this is an R package that uses the very same tokenizers as OpenAI. That is AWESOME. Boast about it!

@JosiahParry
Copy link
Contributor

@CGMossa anything else to add or change on my feedback?

@DavZim
Copy link
Author

DavZim commented Nov 10, 2024

@JosiahParry, I've included a bigger section about another cool use-case and addressed most of the open points.
The only part open is the cover image.

@DavZim
Copy link
Author

DavZim commented Nov 14, 2024

@JosiahParry, is there anything missing from your perspective and do you have any ideas for a cover image?
Otherwise I think this is good to merge.

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