mirror of https://github.com/CGAL/cgal
92 lines
4.0 KiB
Plaintext
92 lines
4.0 KiB
Plaintext
-------------- POUR LA RELEASE
|
|
|
|
- Priorite (sinon on n'est pas 100% backward-compatible) :
|
|
mettre des conversion operators circulator/iterator -> handles.
|
|
et virer les appels a handle() dans la test-suite pour verifier si ca
|
|
marche. Pour les iterators, il faudra surement deriver simplement de
|
|
celui du DS_container.
|
|
- Finir de decider pour find_conflicts, si on veut les facettes du bord du
|
|
trou vues de l'interieur ou de l'exterieur. Implanter ca avant la release !
|
|
(et prevenir David/Raphaelle, les deux seuls users pour le moment).
|
|
- bien tester le output_stream de T_3 et le copy_tds (ordre de creation des
|
|
sommets).
|
|
- Verifier qu'il n'y a pas d'autres operations du meme style a modifier.
|
|
|
|
-------------- POUR DES QU'ON PEUT
|
|
|
|
General
|
|
- constructeurs pour avoir Triangulation T(points.begin(), points.end());
|
|
- mettre des locate_2D, locate_3D, insert idem, pour eviter les tests
|
|
redondants sur la dimension a chaque insertion.
|
|
- trouver une interface avec T_2 qui permette d'eviter de reprogrammer
|
|
tout le 2d dans le 3d
|
|
- insure
|
|
- virer le init() qui n'est plus necessaire, vu qu'on a le default ctor qui
|
|
remarche ?
|
|
- Can we have : typedef Vertex * Vertex_handle; ?
|
|
Problem is const-correctness (see HDS and its Vertex_const_handle...).
|
|
- Virer les fonctions marquees "obsoletes" apres la release.
|
|
|
|
TEST SUITE
|
|
- faire tourner GCOV sur la test-suite, et rajouter ce qu'il faut pour
|
|
atteindre 100%.
|
|
|
|
Doc
|
|
- (optional) sphere et grille en couleur version eps
|
|
- (revoir les anciennes corrections de JF Dufourd)
|
|
- note pour faire remarquer que Delaunay depend de l'ordre d'insertion
|
|
|
|
Exemples - demo
|
|
- en rajouter si bonnes idees...
|
|
- Monique elle veut des exemples pour les output iterators.
|
|
|
|
TDS
|
|
- virer le truc infame dans DS_Container (le magic machin).
|
|
- reflechir a un support de facettes explicites (cf papier de Giesen)
|
|
- comprendre bug dans ancienne version de is_edge de tds, dim 2
|
|
(pb edge_iterator...? etonnant)
|
|
|
|
T3
|
|
- virer push_back (qui n'est deja plus documente, mais il manque des trucs
|
|
dans STL_extensions pour pouvoir le virer completement).
|
|
- .triangle(Facet) devrait retourner un triangle oriente
|
|
(cf mail sur cgal-discuss-l).
|
|
- nettoyer flips/is_flippable (code duplique)
|
|
- incident_facets, incident_cells (vieille demande de Frank a
|
|
rediscuter). Heu, on peut avoir un peu plus de precisions ?
|
|
- is_valid devrait verifier l'enveloppe convexe
|
|
- prevoir differentes sorties : geomview (ameliorer le rendu pour
|
|
donner un effet de perspective), vrml, viewer_3...
|
|
cf Polyhedron_3, file I/O.
|
|
|
|
Delaunay
|
|
- le numero de vertex necessaire au remove devrait etre un requirement de
|
|
Delaunay sur VertexBase, et le compteur, un data member de Delaunay.
|
|
- optimiser remove :
|
|
- algo d'Olivier (?)
|
|
- pour la creation de la TDS_2, on devrait pouvoir faire plus simple,
|
|
sachant que le trou n'est pas quelconque, mais etoile, et donc on peut
|
|
profiter de ca pour avoir les adjacences directement (peut-etre en
|
|
squattant le flag des cellules pour y mettre un pointeur vers la facette
|
|
2d correspondante).
|
|
- For Delaunay, we can use the visibility (non stochastic) walk. How to
|
|
choose ? Is it worth the complexity (benchmark first) ?
|
|
- nearest neighbor
|
|
- natural neighbors de Raphaelle a integrer
|
|
|
|
Regular :
|
|
- la cell_base speciale qui stocke une liste de points caches ou pas (et un
|
|
tag qui dit si oui ou non elle stocke les points caches ou pas), et un
|
|
iterator pour y acceder.
|
|
- remove devrait etre pareil que pour delaunay (a part les points caches a
|
|
re-inserer apres coup).
|
|
- faire marcher la hierarchie avec. Le probleme est qu'on delete des
|
|
vertex, et donc la hierarchie est perdue avec ses pointeurs up/down qui
|
|
pointent dans l'espace, du coup...
|
|
La solution qu'on a discute est :
|
|
- la hierarchy devrait avoir tous les etages superieurs simplement Delaunay,
|
|
et pas autre chose.
|
|
- le insert() (ou find_conflict) de Regular devrait fournir un iterator sur
|
|
les Vertex_handle correspondant aux points qui seront caches, et en
|
|
fonction de ca, une hierarchy speciale Regular fera ce qu'il faut.
|