Ticket #11108 (closed enhancement: fixed)

Opened 2 years ago

Last modified 2 years ago

Improve class_graph

Reported by: nthiery Owned by: jason
Priority: major Milestone: sage-4.7.1
Component: misc Keywords: class, inheritance, graph
Cc: ncohen, sage-combinat Work issues:
Report Upstream: N/A Reviewers: Nathann Cohen
Authors: Nicolas M. Thiéry Merged in: sage-4.7.1.alpha0
Dependencies: Stopgaps:

Description (last modified by nthiery) (diff)

Improvements to class_graph:

  • Returns directly a DiGraph? by default
  • Supports a class or an object as input
  • Improves the doc

Apply :

  • trac_11108-classgraph-nt.patch

Attachments

trac_11108-review.patch Download (2.0 KB) - added by ncohen 2 years ago.
trac_11108-classgraph-nt.patch Download (6.8 KB) - added by nthiery 2 years ago.
Final patch with Nathann's review

Change History

comment:1 Changed 2 years ago by nthiery

  • Status changed from new to needs_review

comment:2 Changed 2 years ago by nthiery

  • Description modified (diff)

Changed 2 years ago by ncohen

comment:3 Changed 2 years ago by ncohen

  • Description modified (diff)

Hello !!!

Here is a small patch adding 3 lines of comments to your code, so that other will understand it faster than I did :-)

If that's fine with you, you can set the ticket to "positive review" !

Nathann

Changed 2 years ago by nthiery

Final patch with Nathann's review

comment:4 Changed 2 years ago by nthiery

  • Reviewers set to Nathann Cohen
  • Description modified (diff)

Hi Nathann,

Thanks for your review! The comments were welcome indeed :-) I rephrased them just a bit, and folded your patch into yours.

No comment on sage-devel, so I guess noone cares.

Cheers,

Nicolas

comment:5 Changed 2 years ago by ncohen

  • Status changed from needs_review to positive_review

Great ! This one's good to go ! I wanted to check the documentation again but Sage does not like to have some file added in a branch that disappear in another :-)

Nathann

comment:6 Changed 2 years ago by jdemeyer

  • Milestone changed from sage-4.7 to sage-4.7.1

comment:7 Changed 2 years ago by jdemeyer

  • Status changed from positive_review to closed
  • Resolution set to fixed
  • Merged in set to sage-4.7.1.alpha0
Note: See TracTickets for help on using tickets.