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

feat(api): add markdown support #95

Closed
wants to merge 1 commit into from
Closed

Conversation

revolunet
Copy link
Contributor

@revolunet revolunet commented Dec 2, 2024

Trying to add Markdown support

The markdown parser is extracted from llamaindex

@revolunet revolunet marked this pull request as ready for review December 4, 2024 09:25
@leoguillaume
Copy link
Contributor

leoguillaume commented Dec 5, 2024

@revolunet Merci beaucoup Julien pour cette contribution, ca me semble nickel :) !

Je m'interroge juste sur la nécessité de maintenir le code parsing, j'ai vu que des libraires semblent très complète et rapide comme : https://github.com/miyuchina/mistletoe/blob/master/performance.md

Qu'en penses-tu ?

Hate de tester le dauphinois au potimarron. On fait quelques migrations en ce moment je merge ca dans les prochains jours !

@revolunet
Copy link
Contributor Author

Hello, possible de faire tourner les tests dans la CI ?

Je plussoie pour utiliser des implems externes quand c'est possible. tu veux que j'essaie dans cette PR ?

@leoguillaume
Copy link
Contributor

Hello, possible de faire tourner les tests dans la CI ?

Je plussoie pour utiliser des implems externes quand c'est possible. tu veux que j'essaie dans cette PR ?

@revolunet C'est un gros chantier qu'on a pas encore entreprit, car il faudrait mock tous les clients externe qui permettent de lancer l'API : vLLM, TEI, Qdrant, Grist ...

Ce qui serait très utile pour les contributeurs également qui n'otn pas accès à des GPUs.

Tu voyais ca comme ca également ?

@leoguillaume
Copy link
Contributor

@revolunet PR migrée dans #106 pour rebase

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