Skip to content

Conversation

@gnestor
Copy link
Contributor

@gnestor gnestor commented Aug 13, 2016

@gnestor
Copy link
Contributor Author

gnestor commented Aug 13, 2016

@poke1024 @takluyver @rgbkrk A couple questions:

  • Are ... delimiters for AsciiMath standard?
  • Do we want to support Experimental mml3?
  • Do we want to support semantic and annotations?
  • What version of notebook/nbviewer is Github running? How often to they update? Will this prove problematic for users if these features are enabled in their local environment but not elsewhere like Github?

@gnestor gnestor mentioned this pull request Aug 13, 2016
@minrk
Copy link
Member

minrk commented Aug 13, 2016

What version of notebook/nbviewer is Github running? How often to they update?

I don't know what version of nbconvert GitHub is using, but they seem to update extremely rarely, if ever, and have not been responsive in particular to longstanding math rendering errors in the version they still have deployed.

Will this prove problematic for users if these features are enabled in their local environment but not elsewhere like Github?

Yes, likely. We can propagate it to nbconvert easily enough, so it can land on nbviewer probably before we release it in the stable version of the live notebook.

Are ... delimiters for AsciiMath standard?

I'm not sure what's standard in AsciiMath, but what's the impetus for supporting asciimath here? The $ delimiters we started with for mathjax have proven pretty troublesome, since any delimiter type thing is a defacto extension of markdown, which gets hairy when it starts interacting with the markdown rendering, and further loosens the definition of what markdown we actually support (since it's no longer markdown once there is math).

@rgbkrk
Copy link
Member

rgbkrk commented Aug 13, 2016

Yeah I'd like to let the MathML play out before doing Ascii Math. I haven't seen much use of it and it seems like development has slowed down. I'd want to wait on extensions to commonmark for further mathiness.

@gnestor
Copy link
Contributor Author

gnestor commented Aug 13, 2016

Ok removed AsciiMath support. Is there any issue with supporting Content MathML?

@rgbkrk
Copy link
Member

rgbkrk commented Aug 14, 2016

Nah, I think this looks good. You'll want to rebase with master now that I merged your constants PR.

@poke1024
Copy link
Contributor

Getting rid of AsciiMath is fine. Changes look good to me.

@gnestor
Copy link
Contributor Author

gnestor commented Aug 16, 2016

I'm having a hard time rebasing this branch and given that the only outstanding change left is Content MathML support, I am closing this PR and submitting a new one: #1685

@gnestor gnestor closed this Aug 16, 2016
@gnestor gnestor deleted the mime-mathml branch August 16, 2016 18:03
@minrk minrk modified the milestone: no action Nov 23, 2016
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants