Ticket #10518 (closed enhancement: fixed)
Add dot2tex as optional spkg
| Reported by: | nthiery | Owned by: | jason, ncohen, rlm |
|---|---|---|---|
| Priority: | major | Milestone: | sage-4.6.2 |
| Component: | packages: optional | Keywords: | graphviz |
| Cc: | sage-combinat | Work issues: | |
| Report Upstream: | N/A | Reviewers: | Pablo Angulo |
| Authors: | Nicolas M. Thiéry | Merged in: | |
| Dependencies: | Stopgaps: |
Description
Title says it all. A candidate spkg is available from #7004.
Change History
comment:2 follow-up: ↓ 3 Changed 2 years ago by pang
- Status changed from needs_review to positive_review
I've inspected the package and it looks fine. I've installed it with no warning.
I've tested it, and it works on the proposed examples. It fails to compile latex plots of some graphs with non-numeric labels, but this is unrelated to the package IMHO. The fix would be to either generate better dot code for those graphs, or configure the latex environment better or in dot2tex itself. I don't really know and will report a bug when I research a bit more.
Finally, testing the library after installing the spkg in a 4.6.1 install that I just updated I got one error that seems completely unrelated (on "devel/sage/sage/tests/cmdline.py"). A fresh install also fails on that test.
I'd say this is enough for a positive review, but be my guest.
comment:3 in reply to: ↑ 2 Changed 2 years ago by nthiery
- Reviewers set to Pablo Angulo
- Milestone set to sage-4.6.2
Hi Pablo!
Thanks for the review!
Replying to pang:
I've inspected the package and it looks fine. I've installed it with no warning.
I've tested it, and it works on the proposed examples. It fails to compile latex plots of some graphs with non-numeric labels, but this is unrelated to the package IMHO. The fix would be to either generate better dot code for those graphs, or configure the latex environment better or in dot2tex itself. I don't really know and will report a bug when I research a bit more.
I am curious! Feel free to send me an example.
