Opened 10 years ago
Closed 10 years ago
#10067 closed defect (fixed)
Another broken docstring in is_even_hole_free
Reported by: | ncohen | Owned by: | jason, ncohen, rlm |
---|---|---|---|
Priority: | major | Milestone: | sage-4.6.1 |
Component: | graph theory | Keywords: | |
Cc: | Merged in: | ||
Authors: | Reviewers: | ||
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
Just casting this BipartiteGraph? to a graph is sufficient.
Nathann
Attachments (1)
Change History (7)
Changed 10 years ago by
comment:1 follow-up: ↓ 4 Changed 10 years ago by
- Status changed from new to needs_review
comment:2 follow-up: ↓ 3 Changed 10 years ago by
comment:3 in reply to: ↑ 2 Changed 10 years ago by
comment:4 in reply to: ↑ 1 Changed 10 years ago by
- Status changed from needs_review to needs_info
Replying to ncohen:
This patch looks pretty weird. It essentially says that methods is_even_hole_free() and is_forest() cannot be applied to an instance of BipartiteGraph?. Why is that? Isn't BipartiteGraph? inheriting from Graph?
IMHO it looks like you cure a symptom rather than the root of the problem.
comment:5 Changed 10 years ago by
You're right. I gave you my best attempt at a reply in #10081.
Nathann
comment:6 Changed 10 years ago by
- Resolution set to fixed
- Status changed from needs_info to closed
Fixed by #10081
Note: See
TracTickets for help on using
tickets.
See comment 20ff at #9925 for a flaky doctest that #9422 and #10067 should fix.