Opened 2 years ago

Last modified 18 months ago

#28894 closed enhancement

Move most important attributes of FaceIterator to a structure. — at Version 10

Reported by: gh-kliem Owned by:
Priority: major Milestone: sage-9.2
Component: geometry Keywords: combinatorial polyhedron, face iterator, f-vector
Cc: jipilab, gh-LaisRast Merged in:
Authors: Jonathan Kliem Reviewers:
Report Upstream: N/A Work issues:
Branch: public/28894-reb4 (Commits, GitHub, GitLab) Commit: 56dcddfb6a4d1210426370309f4f5de707362efe
Dependencies: Stopgaps:

Status badges

Description (last modified by gh-kliem)

This ticket is part of a series of tickets to parallelize the f-vector for polyhedra.

Python classes may not be accessed in nogil. To access important attributes of the class FaceIterator, we move those attributes to a structure, which can be accessed in nogil.

Along the way, the ticket simplifies some doctests that define cython functions.

Change History (10)

comment:1 Changed 2 years ago by gh-kliem

  • Branch set to public/28894
  • Commit set to 25825497cbfc8ba20badc8f73cd10c16a42632d5

New commits:

2582549important attributes of iterator in structure

comment:2 Changed 2 years ago by gh-kliem

  • Status changed from new to needs_review

comment:3 Changed 2 years ago by gh-kliem

  • Branch changed from public/28894 to public/28894-reb
  • Commit changed from 25825497cbfc8ba20badc8f73cd10c16a42632d5 to db3139061c8c9062cee382866b5a5dd28fd8b2b2

New commits:

db31390important attributes of iterator in structure

comment:4 Changed 2 years ago by embray

  • Milestone changed from sage-9.0 to sage-9.1

Ticket retargeted after milestone closed

comment:5 Changed 22 months ago by gh-kliem

  • Branch changed from public/28894-reb to public/28894-reb2
  • Commit changed from db3139061c8c9062cee382866b5a5dd28fd8b2b2 to 3e7f7f8de3a288619b3fa20807ca8429622010d9

New commits:

c3e2cb7important attributes of iterator in structure
3e7f7f8applied changes to simplicity and simpliciality

comment:6 Changed 22 months ago by git

  • Commit changed from 3e7f7f8de3a288619b3fa20807ca8429622010d9 to c0c9a314d21a0bbe53f877034eec27c6d7fbf2df

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

8d933c3important attributes of iterator in structure
c0c9a31applied changes to simplicity and simpliciality

comment:7 Changed 21 months ago by gh-kliem

  • Status changed from needs_review to needs_work

Apply failed.

comment:8 Changed 21 months ago by gh-kliem

  • Branch changed from public/28894-reb2 to public/28894-reb3
  • Commit changed from c0c9a314d21a0bbe53f877034eec27c6d7fbf2df to 6d0644a68801a0ae6957bb63984103270d02414d
  • Description modified (diff)
  • Status changed from needs_work to needs_review

New commits:

a58ffbeimportant attributes of iterator in structure
6d0644asimplification of doctests

comment:9 Changed 20 months ago by mkoeppe

  • Milestone changed from sage-9.1 to sage-9.2

Batch modifying tickets that will likely not be ready for 9.1, based on a review of the ticket title, branch/review status, and last modification date.

comment:10 Changed 20 months ago by gh-kliem

  • Branch changed from public/28894-reb3 to public/28894-reb4
  • Commit changed from 6d0644a68801a0ae6957bb63984103270d02414d to 56dcddfb6a4d1210426370309f4f5de707362efe
  • Description modified (diff)

New commits:

41eb697important attributes of iterator in structure
56dcddfsrc/simplification of doctests
Note: See TracTickets for help on using tickets.