#31990 closed enhancement (fixed)

ConvexSet_base: Add methods an_element, some_elements

Reported by: Matthias Köppe Owned by:
Priority: major Milestone: sage-9.4
Component: geometry Keywords:
Cc: gh-kliem, Travis Scrimshaw, Yuan Zhou Merged in:
Authors: Matthias Koeppe Reviewers: Travis Scrimshaw
Report Upstream: N/A Work issues:
Branch: 1744ffa (Commits, GitHub, GitLab) Commit: 1744ffadb775f6a9f975f07f858a22f19c5efc8b
Dependencies: #31959 Stopgaps:

Status badges

Description

Although subclasses of ConvexSet_base are not necessarily Parents, it will be useful for them to provide an_element and some_elements methods.

Change History (14)

comment:1 Changed 18 months ago by Matthias Köppe

Dependencies: #31959

comment:2 Changed 18 months ago by Matthias Köppe

Branch: u/mkoeppe/convexset_base__add_methods_an_element__some_elements

comment:3 Changed 18 months ago by Matthias Köppe

Authors: Matthias Koeppe
Commit: a8845171bcb84207ef8b56a139067f809ad7a675
Status: newneeds_review

Last 10 new commits:

6ab5677RelativeInterior.is_universe: New
c085d30Polyhedron_base.interior: Handle the empty polyhedron correctly
686d0afPolyhedron_base.product: Add doctest for alias 'cartesian_product'
2b1d108Merge #31919
7323b10ConvexSet_base._test_contains: Only test extension to AA for exact base rings
94e6858RelativeInterior.ambient, ambient_vector_space, is_universe: New
0c9bc94ConvexSet_base: Add default implementations of ambient, ambient_dim; add doctests
7d3ae5cMerge #31959
2b1040cConvexSet_base.{an_element,some_elements}, Polyhedron.some_elements, RelativeInterior.some_elements: New
a884517PolyhedronFace.some_elements: New

comment:4 Changed 18 months ago by git

Commit: a8845171bcb84207ef8b56a139067f809ad7a675b19b3cc61cb8f0aed9a1f80accca1f5025da06f4

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

ce91e44src/sage/geometry/relative_interior.py: Fix doctest output
b19b3ccMerge #31959

comment:5 Changed 18 months ago by Travis Scrimshaw

Reviewers: Travis Scrimshaw

The only little quip I have is that generally in Sage some_elements() returns a list rather than an iterable. It might be good to be consistent about this.

comment:6 Changed 18 months ago by git

Commit: b19b3cc61cb8f0aed9a1f80accca1f5025da06f4ed898e6b23488a2912cb510ef1d7d59a798d51a7

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

f0e7c58ambient_vector_space docstring: Fix bad blocks
200d967ConvexSet_base.ambient doctest: Actually test the method
1c2bef5Merge #31959
ed898e6ConvexSet_base.an_element, some_elements: Go through generator method _some_elements_, make some_elements return a list

comment:7 Changed 18 months ago by Matthias Köppe

Status: needs_reviewneeds_work

comment:8 Changed 18 months ago by git

Commit: ed898e6b23488a2912cb510ef1d7d59a798d51a77152e1fdcdf6de646218a459870e1bbb885337af

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

b3ac0f3ConvexSet_base.an_element, some_elements: Raise NotImplementedError if _some_elements_ is NotImplemented
7152e1fConvexSet_base._test_contains: Test some_elements too

comment:9 Changed 18 months ago by Matthias Köppe

Status: needs_workneeds_review

comment:10 Changed 18 months ago by Travis Scrimshaw

Thanks. Green bot => positive review.

comment:11 Changed 18 months ago by git

Commit: 7152e1fdcdf6de646218a459870e1bbb885337af1744ffadb775f6a9f975f07f858a22f19c5efc8b

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

f02ca28src/sage/geometry/polyhedron/face.py: Remove unused import
1744ffaMerge #31959

comment:12 Changed 18 months ago by Matthias Köppe

Status: needs_reviewpositive_review

Patchbot is green modulo the unrelated sage.misc.package test failure. Merged latest #31959 to get rid of pyflakes' report on an unused import.

comment:13 Changed 18 months ago by Matthias Köppe

Thanks for reviewing!

comment:14 Changed 17 months ago by Volker Braun

Branch: u/mkoeppe/convexset_base__add_methods_an_element__some_elements1744ffadb775f6a9f975f07f858a22f19c5efc8b
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.