Opened 4 years ago

Closed 4 years ago

#17848 closed defect (fixed)

Add permutations_cython to the doc

Reported by: ncohen Owned by:
Priority: major Milestone: sage-6.6
Component: documentation Keywords:
Cc: vdelecroix Merged in:
Authors: Nathann Cohen Reviewers: Vincent Delecroix
Report Upstream: N/A Work issues:
Branch: 5d6d1ee (Commits) Commit: 5d6d1ee547d0a3c5c4528ff01861441ba579e4ff
Dependencies: Stopgaps:

Description

permutations_cython did not appear in the reference manual, and one of its docstring was broken. This branch adds it where it belongs.

Nathann

Change History (6)

comment:1 Changed 4 years ago by ncohen

  • Branch set to public/17848
  • Status changed from new to needs_review

comment:2 Changed 4 years ago by git

  • Commit set to 5d6d1ee547d0a3c5c4528ff01861441ba579e4ff

Branch pushed to git repo; I updated commit sha1. New commits:

5d6d1eetrac #17848: Add permutations_cython to the doc

comment:3 Changed 4 years ago by vdelecroix

  • Reviewers set to Vincent Delecroix
  • Status changed from needs_review to positive_review

Fine fine. I should really make this appear in sage.combinat.gray_codes.

comment:4 follow-up: Changed 4 years ago by ncohen

I do not think that it is used anywhere. Just a forgotten piece of code.

Actually, it may be cool to see the graph of module imports. Would be interesting to see which guys are never called by anybody else. I don't know how to build it, though :-P

Nathann

comment:5 in reply to: ↑ 4 Changed 4 years ago by vdelecroix

Replying to ncohen:

I do not think that it is used anywhere. Just a forgotten piece of code.

Actually, it may be cool to see the graph of module imports. Would be interesting to see which guys are never called by anybody else. I don't know how to build it, though :-P

You can just parse the import in each files. You only have to care about relative vs absolute import. That would be really cool!

Vincent

comment:6 Changed 4 years ago by vbraun

  • Branch changed from public/17848 to 5d6d1ee547d0a3c5c4528ff01861441ba579e4ff
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.