Skip to content

Reconstruction from parallel slices#5609

Draft
sloriot wants to merge 111 commits intoCGAL:mainfrom
sloriot:Reconstruction_from_parallel_slices_3-new_package
Draft

Reconstruction from parallel slices#5609
sloriot wants to merge 111 commits intoCGAL:mainfrom
sloriot:Reconstruction_from_parallel_slices_3-new_package

Conversation

@sloriot
Copy link
Member

@sloriot sloriot commented Apr 11, 2021 •
edited
Loading

Summary of Changes

Reconstruction from parallel slices based on J.-D. Boissonnat, B. Geiger. Three dimensional reconstruction of complex shapes based on the Delaunay triangulation, Biomedical Image Processing and Biomedical Visualization, 964 (1993)

Screenshot from 2021-04-11 09-56-33

Release Management

  • Affected package(s): New package
  • Feature/Small Feature (if any): TODO
  • Link to compiled documentation (obligatory for small feature) wrong link name to be changed
  • License and copyright ownership: GeometryFactory SARL

TODOs

  • use FaceGraph API
  • the code contains several switches that tweak the algorithm to handle non-manifold issues. We should decide which version should be kept and clean the code
  • Clean up/Optimization
  • Doc
  • Tests
  • API for dump 3D tetmesh

sloriot and others added 30 commits June 19, 2012 16:25
@sloriot
new branch for reconstruction of parallel slices
8b0944a
@sloriot
move from experimental-packages to feature-branch
1744171
@sloriot
add package info
39065af
@sloriot
move cpp to examples
5ccdbf7
@sloriot
forgot to reproject the point in that case
8df1286
@sloriot
wrong location
d615f7b
@sloriot
merge from next
69c3891
@sloriot
bad comment
3fb409b
@sloriot
hide warnings
adefbfc
@sloriot
writer clear polyhedron before reading it
5f6bdac
@sloriot
update writer to be more generic
67dda1b
@sloriot
first version of reconstruction plugin
623b22a
@sloriot
merge from next
1bf277f
@sloriot
only clear it once
76c687f
@sloriot
forgot I already had a debug macro
cdb0285
@sloriot
the index of vertices in cdt is compulsary and must not be done only …
9cecf13 
…for debug!
@sloriot
add a visitor inside the slice writer to indicate when a layer is ready
0deb4cd
@sloriot
add a visitor to update the display while reconstructing the model
5db2a21
@sloriot
merge from next
cc74355
@sloriot
static const double cannot be init in class
0c69835
@sloriot
merge from next
e77dff0
@sloriot
remove a todo but add another one
54b94d2
@sloriot
fix elevation
3cc494b
@sloriot
add todo
d2240d8
@sloriot
adding some data files
081c3b6
@sloriot
add a script for testing code on a set of input
0cc221a
@sloriot
create res dir
08fbf66
@sloriot
Merge branch 'hot-fixes-for-Git' into Reconstruction_parallel_slices-…
bc2699a 
…new_package-sloriot

Conflicts:
	.gitattributes

I solved it by replacing it by the one of master
@sloriot
crlf to lf eol recoding
844256a
@sloriot
remove unused int
c809127
@sloriot sloriot added Feature Not yet approved The feature or pull-request has not yet been approved. TODO labels Apr 11, 2021
@sloriot sloriot changed the title Reconstruction from parallel slices 3 new package [Feature] Reconstruction from parallel slices Apr 11, 2021
@sloriot sloriot changed the title [Feature] Reconstruction from parallel slices Reconstruction from parallel slices Apr 11, 2021
@sloriot sloriot added this to the 5.5-beta1 milestone Sep 23, 2021
@MaelRL MaelRL modified the milestones: 5.5-beta, 5.6-beta Mar 28, 2022
@MaelRL MaelRL modified the milestones: 5.6-beta, 5.7-beta Mar 23, 2023
@janetournois janetournois modified the milestones: 6.0-beta, 6.1-beta May 16, 2024
@MaelRL MaelRL modified the milestones: 6.1-beta, 6.2-beta Mar 17, 2025
@sloriot sloriot changed the base branch from master to main September 16, 2025 19:20
@sloriot sloriot mentioned this pull request Dec 22, 2025
Open
@MaelRL MaelRL modified the milestones: 6.2-beta, 6.3-beta Mar 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Feature Not yet approved The feature or pull-request has not yet been approved. TODO

Projects

None yet

Milestone

6.3-beta

Development

Successfully merging this pull request may close these issues.

6 participants

@sloriot @MaelRL @janetournois @afabri @lrineau @maxGimeno