Opened 11 months ago
Closed 11 months ago
#31990 closed enhancement (fixed)
ConvexSet_base: Add methods an_element, some_elements
Reported by: | mkoeppe | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | sage-9.4 |
Component: | geometry | Keywords: | |
Cc: | gh-kliem, tscrim, yzh | Merged in: | |
Authors: | Matthias Koeppe | Reviewers: | Travis Scrimshaw |
Report Upstream: | N/A | Work issues: | |
Branch: | 1744ffa (Commits, GitHub, GitLab) | Commit: | 1744ffadb775f6a9f975f07f858a22f19c5efc8b |
Dependencies: | #31959 | Stopgaps: |
Description
Although subclasses of ConvexSet_base
are not necessarily Parent
s, it will be useful for them to provide an_element
and some_elements
methods.
Change History (14)
comment:1 Changed 11 months ago by
- Dependencies set to #31959
comment:2 Changed 11 months ago by
- Branch set to u/mkoeppe/convexset_base__add_methods_an_element__some_elements
comment:3 Changed 11 months ago by
- Commit set to a8845171bcb84207ef8b56a139067f809ad7a675
- Status changed from new to needs_review
comment:4 Changed 11 months ago by
- Commit changed from a8845171bcb84207ef8b56a139067f809ad7a675 to b19b3cc61cb8f0aed9a1f80accca1f5025da06f4
comment:5 Changed 11 months ago by
- Reviewers set to 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 11 months ago by
- Commit changed from b19b3cc61cb8f0aed9a1f80accca1f5025da06f4 to ed898e6b23488a2912cb510ef1d7d59a798d51a7
Branch pushed to git repo; I updated commit sha1. New commits:
f0e7c58 | ambient_vector_space docstring: Fix bad blocks
|
200d967 | ConvexSet_base.ambient doctest: Actually test the method
|
1c2bef5 | Merge #31959
|
ed898e6 | ConvexSet_base.an_element, some_elements: Go through generator method _some_elements_, make some_elements return a list
|
comment:7 Changed 11 months ago by
- Status changed from needs_review to needs_work
comment:8 Changed 11 months ago by
- Commit changed from ed898e6b23488a2912cb510ef1d7d59a798d51a7 to 7152e1fdcdf6de646218a459870e1bbb885337af
comment:9 Changed 11 months ago by
- Status changed from needs_work to needs_review
comment:10 Changed 11 months ago by
Thanks. Green bot => positive review.
comment:11 Changed 11 months ago by
- Commit changed from 7152e1fdcdf6de646218a459870e1bbb885337af to 1744ffadb775f6a9f975f07f858a22f19c5efc8b
comment:12 Changed 11 months ago by
- Status changed from needs_review to positive_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 11 months ago by
Thanks for reviewing!
comment:14 Changed 11 months ago by
- Branch changed from u/mkoeppe/convexset_base__add_methods_an_element__some_elements to 1744ffadb775f6a9f975f07f858a22f19c5efc8b
- Resolution set to fixed
- Status changed from positive_review to closed
Note: See
TracTickets for help on using
tickets.
Last 10 new commits:
RelativeInterior.is_universe: New
Polyhedron_base.interior: Handle the empty polyhedron correctly
Polyhedron_base.product: Add doctest for alias 'cartesian_product'
Merge #31919
ConvexSet_base._test_contains: Only test extension to AA for exact base rings
RelativeInterior.ambient, ambient_vector_space, is_universe: New
ConvexSet_base: Add default implementations of ambient, ambient_dim; add doctests
Merge #31959
ConvexSet_base.{an_element,some_elements}, Polyhedron.some_elements, RelativeInterior.some_elements: New
PolyhedronFace.some_elements: New