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

Add see also section to integrate over a polytope #22824

Closed
mforets mannequin opened this issue Apr 18, 2017 · 8 comments
Closed

Add see also section to integrate over a polytope #22824

mforets mannequin opened this issue Apr 18, 2017 · 8 comments
Labels
c: documentation p: major / 3 t: enhancement
Milestone
sage-8.0

Comments

@mforets
Copy link
Mannequin

mforets mannequin commented Apr 18, 2017

Add a "seealso::" section in the doc of the integrate function to point to that method of polytopes.

CC: @sagetrac-tmonteil

Component: documentation

Keywords: polyhedron, integrate

Author: Marcelo Forets

Branch/Commit: 6bed290

Reviewer: Travis Scrimshaw

Issue created by migration from https://trac.sagemath.org/ticket/22824
@mforets mforets mannequin added this to the sage-8.0 milestone Apr 18, 2017
@mforets mforets mannequin added c: documentation labels Apr 18, 2017
@mforets
Copy link
Mannequin Author

mforets mannequin commented Apr 18, 2017

comment:1

(for reference) A more ambitious task would be to modify the integrate function with an additional elif statement to handle integrate(polynomial,polytope).

@mforets
Copy link
Mannequin Author

mforets mannequin commented Apr 18, 2017

Branch: u/mforets/add_see_also_section_to_integrate_over_a_polytope

@mforets
Copy link
Mannequin Author

mforets mannequin commented Apr 18, 2017

Commit: 6bed290

@mforets
Copy link
Mannequin Author

mforets mannequin commented Apr 18, 2017

New commits:

6bed290add see also block

@mforets mforets mannequin added the s: needs review label Apr 18, 2017
@tscrim
Copy link
Collaborator

tscrim commented Apr 18, 2017

Reviewer: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented Apr 18, 2017

comment:4

Once you add your name to the authors block, you can set a positive review.

@mforets
Copy link
Mannequin Author

mforets mannequin commented Apr 18, 2017

Author: Marcelo Forets

@vbraun
Copy link
Member

vbraun commented Apr 23, 2017

Changed branch from u/mforets/add_see_also_section_to_integrate_over_a_polytope to 6bed290

@vbraun vbraun closed this as completed in 8058dca Apr 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: documentation p: major / 3 t: enhancement
Projects
None yet
Milestone
sage-8.0
Development

No branches or pull requests
2 participants
@vbraun @tscrim