Skip to content

Conversation

@jakubnowicki
Copy link
Member

Basic vignette for using LLM tools with Rhino.

How to test?

Run pkgdown::build_site() to preview the documentation page.

Copy link
Contributor

@pstorozenko pstorozenko left a comment

Choose a reason for hiding this comment

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

Looks good. My only concern is that the UX of adding instructions to your project is suboptimal, hence I added link to documentation with markdown file as suggestion. We could consider adding function rhino::setup_llm_instructions({curosor|copilot}) in the future rhino version.

jakubnowicki and others added 2 commits April 1, 2025 12:21
Co-authored-by: Piotr Pasza Storożenko <pitrosk@gmail.com>
Co-authored-by: Piotr Pasza Storożenko <pitrosk@gmail.com>
@jakubnowicki
Copy link
Member Author

Looks good. My only concern is that the UX of adding instructions to your project is suboptimal, hence I added link to documentation with markdown file as suggestion. We could consider adding function rhino::setup_llm_instructions({curosor|copilot}) in the future rhino version.

Thanks for those changes. Yes, function(s) for adding instructions are something I would like in the future, once we have a more ready version of those.

@jakubnowicki jakubnowicki merged commit 60c7abd into main Apr 1, 2025
10 checks passed
@jakubnowicki jakubnowicki deleted the using-llm-tools-with-rhino branch April 1, 2025 10:57
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.

3 participants