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

Ensuring proper citation and hyperlink creation for each message #68

Merged
merged 19 commits into from
Dec 8, 2023

Conversation

rohan-uiuc
Copy link
Collaborator

No description provided.

Copy link

vercel bot commented Nov 20, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
uiuc-chat ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 8, 2023 6:31pm

@rohan-uiuc
Copy link
Collaborator Author

We need another PR for ai-ta-backend to update the same prompt for correct token usage count.

@KastanDay
Copy link
Collaborator

Woah it's looking fantastic after this update. I think it's.... perfect?
CleanShot 2023-12-07 at 13 43 59

Copy link
Collaborator

@KastanDay KastanDay left a comment

Choose a reason for hiding this comment

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

Tested and working perfectly. Incredible work.

@KastanDay
Copy link
Collaborator

Great results:
CleanShot 2023-12-08 at 12 10 09
CleanShot 2023-12-08 at 12 12 03
It's legit working, with no hits from me about what disease it is. Even the sources are actually so good, it links to really helpful articles.

@KastanDay KastanDay merged commit 760e6b6 into main Dec 8, 2023
3 checks passed
@KastanDay KastanDay deleted the citations branch December 8, 2023 20:24
hanlily666 pushed a commit that referenced this pull request Jan 23, 2024
… of citations at end of message (#68)

* Ensuring proper citation and hyperlink creation for each message

* Styling link component inside React Markdown (doesn't really work)

* Fixed link styling for citations

* Fix: Handle non-JSON '[DONE]' response in OpenAIStream function to prevent JSON parsing error while streaming the response

* Removed debugging log

* Removed arrows from footnotes and styled sup and footnote links to go with the web.

* Styling changes to footnotes

* Clickable citation links with styling

* Styled the sources better, handled other buttons in React Markdown component

* Improve: Adding index to citations to track the documents better

* Styling fixes

* Minor prompt change to handle the use case with no sources.

* Fix: Styling change to handle other components' styling affected by list of sources.

* Minor prompt change and handling never ending streaming response.

* Minor styling bugfix

* Improve: Changed Img > Text prompt to make it keyword rich for precise retrieval

* Fix: HandleRegenerate fix for messages not containing img description.

* Fix: Handling cases where the citation link is not present, to render efficiently.
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