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

ValueError + AttributeError when calling layout_graphviz on a Hasse diagram #12364

Closed
seblabbe opened this issue Jan 26, 2012 · 13 comments
Closed

ValueError + AttributeError when calling layout_graphviz on a Hasse diagram #12364

seblabbe opened this issue Jan 26, 2012 · 13 comments
Assignees
@sagetrac-sage-combinat
Labels
c: combinatorics p: major / 3 t: bug
Milestone
sage-7.4

Comments

@seblabbe
Copy link
Contributor

With sage-4.7.2 and sage -i dot2tex installed :

sage: m = WordMorphism('a->abb,b->ba')
sage: w = m.fixed_point('a')
sage: prefix = Word(list(w[:100]))
sage: pals = prefix.palindromes()
sage: poset = Poset((pals, lambda x,y: x.is_factor(y)))
sage: H = poset.hasse_diagram()
sage: H.layout_graphviz()         #require dot2tex
ERROR    Failed to process input
Traceback (most recent call last):
...
ValueError: invalid literal for int() with base 10: '249.88'
-------------------------------------------------------
Traceback (most recent call last):
...
AttributeError: 'str' object has no attribute 'iteritems'

This came up in #12351.

CC: @sagetrac-sage-combinat

Component: combinatorics

Keywords: dot2tex

Author: Sébastien Labbé

Branch/Commit: e80f38b

Reviewer: Travis Scrimshaw

Issue created by migration from https://trac.sagemath.org/ticket/12364
@seblabbe seblabbe added this to the sage-5.11 milestone Jan 26, 2012
@seblabbe

This comment has been minimized.

Sign in to view
@nthiery
Copy link
Contributor

nthiery commented Jan 26, 2012

comment:2

Could this be related to http://groups.google.com/group/sage-combinat-devel/msg/13cd26ebb99954db

@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.2, sage-6.3 May 6, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.3, sage-6.4 Aug 10, 2014
@fchapoton
Copy link
Contributor

comment:7

works in 7.3.b7

@fchapoton fchapoton modified the milestones: sage-6.4, sage-7.3 Jul 11, 2016
@tscrim
Copy link
Collaborator

tscrim commented Jul 11, 2016

Reviewer: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented Jul 11, 2016

comment:8

Agreed.

@tscrim tscrim removed this from the sage-7.3 milestone Jul 11, 2016
@seblabbe
Copy link
Contributor Author

comment:10

Shouldn't we add a doctest?

@tscrim
Copy link
Collaborator

tscrim commented Jul 12, 2016

comment:11

Go for it.

@seblabbe
Copy link
Contributor Author

Branch: u/slabbe/12364

@seblabbe
Copy link
Contributor Author

Commit: e80f38b

@seblabbe
Copy link
Contributor Author

comment:12

Sure!

New commits:

e80f38b12364: adding doctest

@tscrim
Copy link
Collaborator

tscrim commented Aug 22, 2016

Author: Sébastien Labbé

@tscrim
Copy link
Collaborator

tscrim commented Aug 22, 2016

comment:13

Thanks.

@tscrim tscrim added this to the sage-7.4 milestone Aug 22, 2016
@vbraun
Copy link
Member

vbraun commented Aug 24, 2016

Changed branch from u/slabbe/12364 to e80f38b

@vbraun vbraun closed this as completed in 6b45951 Aug 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sagetrac-sage-combinat sagetrac-sage-combinat

Labels
c: combinatorics p: major / 3 t: bug
Projects
None yet
Milestone
sage-7.4
Development

No branches or pull requests
6 participants
@nthiery @vbraun @tscrim @seblabbe @jdemeyer @fchapoton