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.

Sign up for GitHub

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

Jump to bottom

Support GinacFunction._print_latex_() customization #20888

Closed
rwst opened this issue Jun 27, 2016 · 9 comments
Closed

Support GinacFunction._print_latex_() customization #20888

rwst opened this issue Jun 27, 2016 · 9 comments
Labels
c: symbolics r: wontfix t: enhancement

Comments

@rwst
Copy link

rwst commented Jun 27, 2016

Defining a LaTeX print function (instead of just a latex name) is only possible for GinacFunctions by changing Pynac.

I think it would be more consistent to simply allow the same mechanism as with BuiltinFunctions. This should pose no problem, since there it is already channeled through Pynac. Effectively, if there is not a LaTeX print function defined in Pynac then the Python _print_latex_ member should be tried.

This s pynac/pynac#184

See also #11428

CC: @paulmasson

Component: symbolics

Issue created by migration from https://trac.sagemath.org/ticket/20888
@rwst rwst added this to the sage-7.3 milestone Jun 27, 2016
@rwst

This comment has been minimized.

Sign in to view
@rwst

This comment has been minimized.

Sign in to view
@rwst
Copy link
Author

rwst commented Jun 27, 2016

comment:3

Implemented in pynac git master.

@paulmasson
Copy link
Mannequin

paulmasson mannequin commented Jul 20, 2016

Dependencies: #21034

@paulmasson paulmasson mannequin added the s: needs info label Jul 20, 2016
@rwst
Copy link
Author

rwst commented Jul 20, 2016

comment:5

What was the question?

@paulmasson
Copy link
Mannequin

paulmasson mannequin commented Jul 20, 2016

comment:6

Waiting for pynac 0.6.8

@rwst
Copy link
Author

rwst commented Aug 11, 2016

comment:7

Implemented in #21034.

@rwst
Copy link
Author

rwst commented Aug 11, 2016

Changed dependencies from #21034 to none

@rwst rwst removed this from the sage-7.3 milestone Aug 11, 2016
@embray
Copy link
Contributor

embray commented Aug 30, 2016

comment:8

Determined to be invalid/duplicate/wontfix (closing as "wontfix" as a catch-all resolution).

@embray embray closed this as completed Aug 30, 2016
@rwst rwst mentioned this issue Aug 7, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: symbolics r: wontfix t: enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@embray @rwst