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 PDF to Markdown feature #442

Closed
wants to merge 0 commits into from

Conversation

julien-blanchon
Copy link
Contributor

This pull request introduces the PDF to Markdown conversion. This enhancement aims to provide users with a seamless experience when discussing and referencing content from PDFs, especially in academic and research contexts.

@nsarrazin
Copy link
Collaborator

Very cool idea! I'll do a deeper review later but thanks for contributing already 😄

@julien-blanchon
Copy link
Contributor Author

Very cool idea! I'll do a deeper review later but thanks for contributing already 😄

Great, I write a more detailed issues about this here.

It seems that I got some problem with my fork branch, look like I wasn't update at time of the PR.
I will fix that soon.

If you figure out any inconsistencies with the code base or just typescript/svelte bad practice feel free to comment or propose a fix.
This features was originaly code for my personal usage but I think this could be usefull for everyone, in particular for research paper this worflow of converting the file to native markdown/latex give impressive result with GPT4 and Claude2.

I also did a PR about adding the OpenAI API support in Chat UI here.

@julien-blanchon
Copy link
Contributor Author

@hungryalgo
#449 is outdated. Yes, having the PDF in the chat interface would be very cool. As far as I know, there are no plans for that yet. However, I have suggested the idea of doing a basic PDF integration in chat-ui (without the Mathpix stuff): #531 (comment).

I don't know if there are any plans to add the PDF soon, @nsarrazin might be the best person to ask

@jackbravo
Copy link

Closed in favor of PR #641 that closes issue #609, right?

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