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

Add citations #58

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

Add citations #58

wants to merge 4 commits into from

Conversation

galopyz
Copy link

@galopyz galopyz commented Feb 4, 2025

Hi, @ncoop57

Adding an example to use citations in claudette as stated in #53.
I am still new to git, claudette, and open source projects, so I might have messed up something.

I only provided an example to use plain text. PDF takes many tokens, but if we want to have a PDF example, I could find a small one and show how it can be used.
When using citations, Claude responds with alternating text and citation blocks. The response doesn't look pretty because only the first text block is shown without any citation blocks.

Please let me know if I need to make any corrections.
Thanks.

Copy link

gitnotebooks bot commented Feb 4, 2025

Found 1 changed notebook. Review the changes at https://app.gitnotebooks.com/AnswerDotAI/claudette/pull/58

@ncoop57
Copy link
Contributor

ncoop57 commented Feb 5, 2025

Thanks @galopyz for this PR! its coming together nicely 🤓 How about we work on getting the response to look better especially when it does a lot of citations? If you look at Simon Willison's blog post on it, he has a little section on rendering these citations: https://simonwillison.net/2025/Jan/24/anthropics-new-citations-api/#rendering-the-citations. How about you see if you could add something similar?

@galopyz
Copy link
Author

galopyz commented Feb 7, 2025

Hi, @ncoop57
I have added those changes to display messages with citations as block quotes.
I have merged my branch from AnswerDotAI:main because mine was behind, and there are many changes in gitnotebooks. Should I revert this change?

Let me know if anything needs to be changed. And should I run the whole notebooks? To make sure I didn't break anything? I tried my best to follow coding style and not break anything.

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