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

Support MathJax #8

Closed
0x6b opened this issue Jul 31, 2018 · 9 comments · Fixed by #9
Closed

Support MathJax #8

0x6b opened this issue Jul 31, 2018 · 9 comments · Fixed by #9

Comments

@0x6b
Copy link
Owner

0x6b commented Jul 31, 2018

From #1 (comment).

@0x6b
Copy link
Owner Author

0x6b commented Aug 1, 2018

@allenyllee, I have implemented preliminary MathJax support at commit 2737095. Since I'm not a MathJax guy, could you please test this beta on your end? I only tested with this page which you pointed out.

If you have any issue please let me know the URL with exact selection (screenshots would be appreciated as this is important for debugging). Thanks.

@0x6b
Copy link
Owner Author

0x6b commented Aug 2, 2018

new beta (0.4.1) is available.

@allenyllee
Copy link

allenyllee commented Aug 2, 2018

@0x6b Awesome! very thanks! I think this is the first addon on firefox to support copy as markdown for MathJax. But I still find some bugs, here is the list:

HackMD is an online editor which support Markdown and Mathjax. I have created a locked page to paste the origin and expected results, you can not modify it, but can create a new one if you like.
https://hackmd.io/5UH_EAtTTv-QA7N-n4qIcA?both

issue1:

derivative - Backpropagation with Softmax / Cross Entropy - Cross Validated

unexpected line break

snipaste_2018-08-02_12-33-32
snipaste_2018-08-02_12-34-42

expected

snipaste_2018-08-02_12-33-56
snipaste_2018-08-02_12-35-27

issue2:

LDA(Latent Dirichlet Allocation)主题模型 - CSDN博客

unexpected messed list

snipaste_2018-08-02_12-35-54

expected

snipaste_2018-08-02_12-36-29

issue3:

卷积神经网络(CNN)在句子建模上的应用 | Jey Zhang

unexpected space

snipaste_2018-08-02_12-37-18

expected

snipaste_2018-08-02_12-51-32

issue4

LDA(Latent Dirichlet Allocation)主题模型 - CSDN博客

unexpected messed list(multi-level)

snipaste_2018-08-02_12-38-45

expected

snipaste_2018-08-02_12-39-12

issue5

LDA(Latent Dirichlet Allocation)主题模型 - CSDN博客

unexpected: these lines can not be copied

expected

LDA数学分析

上一小节,我们忽略了一个问题,就是如何确定主题和词汇分布,还有文档与词汇的分布,但是要搞明白这个问题,就避免不了一些数学分析了。再次强烈推荐rickjin的《LDA数学八卦》还有Gregor Heinrich有著名的《Parameter estimation for text analysis》。此处我只做一些关键扼要的理论推导 :-)

多项式分布

@0x6b
Copy link
Owner Author

0x6b commented Aug 2, 2018

@allenyllee much appreciate your efforts! Let me see.

0x6b added a commit that referenced this issue Aug 2, 2018
Also implement paragraphBeforeScriptNode rule to fix part of #8 (issue1).
0x6b added a commit that referenced this issue Aug 3, 2018
@0x6b
Copy link
Owner Author

0x6b commented Aug 4, 2018

issue2 seems to be difficult to fix with current addon implementation due to unexpected and ugly DOM structure. Still finding a way to mitigate the situation.

0x6b added a commit that referenced this issue Aug 4, 2018
@allenyllee
Copy link

Hi, could you merge this version (0.4.1) into current version (0.5.2)? Or I have to install these two versions, one for mathjax and one for update. Thanks!

@0x6b
Copy link
Owner Author

0x6b commented Dec 13, 2018

Alright @allenyllee. I will release this initial mathjax support as new release (possibly opt-in feature). Give me some time as I'm extreme busy recently. Thanks for your patience!

0x6b added a commit that referenced this issue Dec 17, 2018
Also implement paragraphBeforeScriptNode rule to fix part of #8 (issue1).
0x6b added a commit that referenced this issue Dec 17, 2018
0x6b added a commit that referenced this issue Dec 17, 2018
@0x6b 0x6b closed this as completed in #9 Dec 17, 2018
@0x6b
Copy link
Owner Author

0x6b commented Dec 17, 2018

Hi @allenyllee, thank you for your patience. I've released v0.6.0. Grab the update from your Add-on manager, check Enable experimental MathJax support, then hit Save to enable this. Thanks!

@allenyllee
Copy link

Finally, it is very handy for me, very thanks!

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 a pull request may close this issue.

2 participants