-
Notifications
You must be signed in to change notification settings - Fork 92
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: convert latex to plain text #91
Conversation
cee74a9
to
f0c87c1
Compare
Hi @ldelossa This looks great, but it seems to break markdown answers. For example "write a python fibbonaci recursive example" gives me vs (the old behavior) so the I'm happy to have a look/play around myself if you don't have time, as I would like very much to include the latex option, but will probably only have time this weekend. |
Uh oh! Should have tested a bit more. I wonder if simply doing the latex conversion after markdown parsing would help. |
Ah, I see a bit of an issue, the markdown processing turns things right into term codes for the console. Where to interject latex processing then is a bit tougher. 🤔 |
f0c87c1
to
e5cb1a2
Compare
@npiv latest push seems to fix markdown parsing in our simple testing here. I saw that the latex parsing would change '``' sequences to smart quotes. We just swap them back now. |
Newest GPT models love to provide latex equations. Unfortunately this makes things pretty impossible to read for larger questions. Add a conversion to change latex markup to plain text. Signed-off-by: ldelossa <[email protected]>
e5cb1a2
to
3227590
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah looks good, go ahead. I will release it and bump homebrew tomorrow.
Thanks a lot
Thanks a ton. While I use it, if i find any edge cases ill make more touch up prs that fix weird rendering. Thanks again for chatblade, i literally use this 100 times a day. |
Just a heads up, I can't hit merge cuz I don't have write access :) |
Hey @npiv was this ever released? Got a new laptop today and installed latest chatblade, but I don't think this is there. |
Hmm something went wrong, should be fixed now at least on pypi if you install via pip. Homebrew will follow, but they can take a few hours usually |
@npiv thanks! just upgraded with pip, working :) |
Newest GPT models love to provide latex markup.
Unfortunately this makes things pretty impossible to read for larger questions.
Add a conversion to change latex markup to plain text.
Closes #87
Before:
After: