Opened 6 years ago

Closed 5 years ago

#15047 closed defect (fixed)

LaTeX typeseting of SR.wild should prefix $ with \

Reported by: aginiewicz Owned by:
Priority: minor Milestone: sage-6.7
Component: symbolics Keywords: pynac wildcard latex
Cc: Merged in:
Authors: Ralf Stephan Reviewers: Jan Keitel
Report Upstream: Fixed upstream, in a later stable release. Work issues:
Branch: ef13c36 (Commits) Commit: ef13c3612267b85c5b79e77298b4f76368b7d746
Dependencies: #18237 Stopgaps:

Description

Currently calling

latex(SR.wild(0))

results in '$0'. When using this with SageTeX, it causes issues due to special meaning of $ in TeX. I believe this call should return '\$0' instead. For now when using SageTeX I have to use something like

LatexExpr(latex(SR.wild(0)).replace(r'$',r'\$'))

Change History (16)

comment:1 Changed 6 years ago by aginiewicz

I will provide patch for it later this month when I have time, unless someone does it earlier, as this should be trivial to fix (like single keystroke).

comment:2 Changed 6 years ago by aginiewicz

  • Keywords pynac added

comment:3 Changed 6 years ago by aginiewicz

  • Priority changed from trivial to minor

comment:4 Changed 6 years ago by aginiewicz

Right now I'm not sure if I will provide patch this month, because I wasn't aware it comes from pynac - I've never looked into pynac... yet.

comment:5 Changed 6 years ago by vbraun_spam

  • Milestone changed from sage-6.1 to sage-6.2

comment:6 Changed 6 years ago by vbraun_spam

  • Milestone changed from sage-6.2 to sage-6.3

comment:7 Changed 5 years ago by vbraun_spam

  • Milestone changed from sage-6.3 to sage-6.4

comment:8 Changed 5 years ago by rws

  • Dependencies set to #18155
  • Milestone changed from sage-6.4 to sage-6.6
  • Report Upstream changed from N/A to Fixed upstream, in a later stable release.

comment:9 Changed 5 years ago by rws

  • Branch set to u/rws/latex_typeseting_of_sr_wild_should_prefix___with__

comment:10 Changed 5 years ago by rws

  • Authors set to Ralf Stephan
  • Commit set to 03070400f6175b16c6d94541b5a410048eadbb09
  • Status changed from new to needs_review

New commits:

030704015047: doctest; fixed in pynac-0.3.3

comment:11 Changed 5 years ago by behackl

  • Dependencies changed from #18155 to #18237

comment:12 Changed 5 years ago by jkeitel

I'd be happy to give this a positive review, but I think you made a typo - this ticket number is 15047, not 15048. ;-)

comment:13 Changed 5 years ago by git

  • Commit changed from 03070400f6175b16c6d94541b5a410048eadbb09 to ef13c3612267b85c5b79e77298b4f76368b7d746

Branch pushed to git repo; I updated commit sha1. New commits:

ef13c3615047: typo

comment:14 Changed 5 years ago by jkeitel

  • Reviewers set to Jan Keitel
  • Status changed from needs_review to positive_review

Checked with new pynac version and LGTM

comment:15 Changed 5 years ago by rws

  • Milestone changed from sage-6.6 to sage-6.7

Thanks.

comment:16 Changed 5 years ago by vbraun

  • Branch changed from u/rws/latex_typeseting_of_sr_wild_should_prefix___with__ to ef13c3612267b85c5b79e77298b4f76368b7d746
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.