Opened 8 years ago
Closed 6 years ago
#15047 closed defect (fixed)
LaTeX typeseting of SR.wild should prefix $ with \
Reported by:  aginiewicz  Owned by:  

Priority:  minor  Milestone:  sage6.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, GitHub, GitLab)  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 8 years ago by
comment:2 Changed 8 years ago by
 Keywords pynac added
comment:3 Changed 8 years ago by
 Priority changed from trivial to minor
comment:4 Changed 8 years ago by
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 7 years ago by
 Milestone changed from sage6.1 to sage6.2
comment:6 Changed 7 years ago by
 Milestone changed from sage6.2 to sage6.3
comment:7 Changed 7 years ago by
 Milestone changed from sage6.3 to sage6.4
comment:8 Changed 6 years ago by
 Dependencies set to #18155
 Milestone changed from sage6.4 to sage6.6
 Report Upstream changed from N/A to Fixed upstream, in a later stable release.
comment:9 Changed 6 years ago by
 Branch set to u/rws/latex_typeseting_of_sr_wild_should_prefix___with__
comment:10 Changed 6 years ago by
 Commit set to 03070400f6175b16c6d94541b5a410048eadbb09
 Status changed from new to needs_review
New commits:
0307040  15047: doctest; fixed in pynac0.3.3

comment:11 Changed 6 years ago by
 Dependencies changed from #18155 to #18237
comment:12 Changed 6 years ago by
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 6 years ago by
 Commit changed from 03070400f6175b16c6d94541b5a410048eadbb09 to ef13c3612267b85c5b79e77298b4f76368b7d746
Branch pushed to git repo; I updated commit sha1. New commits:
ef13c36  15047: typo

comment:14 Changed 6 years ago by
 Reviewers set to Jan Keitel
 Status changed from needs_review to positive_review
Checked with new pynac version and LGTM
comment:16 Changed 6 years ago by
 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
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).