Opened 13 months ago
Closed 12 months ago
#27009 closed enhancement (fixed)
py3: avoid sorting vertices and edges in method treewidth
| Reported by: | dcoudert | Owned by: | |
|---|---|---|---|
| Priority: | major | Milestone: | sage-8.7 |
| Component: | graph theory | Keywords: | py3, graph |
| Cc: | tscrim, chapoton | Merged in: | |
| Authors: | David Coudert | Reviewers: | Travis Scrimshaw |
| Report Upstream: | N/A | Work issues: | |
| Branch: | f830eb9 (Commits) | Commit: | f830eb94a953905c3b6e6540e46e2319fb8a3e41 |
| Dependencies: | Stopgaps: |
Description
We avoid sorting vertices, edges and connected components. The code is better this way, and this fixes a doctest error with py3.
Change History (7)
comment:1 Changed 13 months ago by
- Branch set to public/27009_treewidth
- Cc tscrim chapoton added
- Commit set to f830eb94a953905c3b6e6540e46e2319fb8a3e41
- Status changed from new to needs_review
comment:2 Changed 13 months ago by
Why do you use for v in G.vertices(sort=False): instead of one of the iterators?
Other than that LGTM.
comment:3 Changed 13 months ago by
The graph is modified inside the loop over all vertices, so I cannot use an iterator here (at least it's safer this way).
So we can use either G.vertices(sort=False) or list(G).
comment:4 Changed 13 months ago by
- Reviewers set to Travis Scrimshaw
- Status changed from needs_review to positive_review
Okay, thanks for the explanation. LGTM.
comment:5 Changed 13 months ago by
Thank you for the reviews and happy new year !
comment:6 Changed 12 months ago by
- Milestone changed from sage-8.6 to sage-8.7
Retarging tickets optimistically to the next milestone. If you are responsible for this ticket (either its reporter or owner) and don't believe you are likely to complete this ticket before the next release (8.7) please retarget this ticket's milestone to sage-pending or sage-wishlist.
comment:7 Changed 12 months ago by
- Branch changed from public/27009_treewidth to f830eb94a953905c3b6e6540e46e2319fb8a3e41
- Resolution set to fixed
- Status changed from positive_review to closed
New commits:
trac #27009: avoid sorting in method treewidth