Skip to content

IO: add support for VG (VertexGroup) format#9436

Open
soesau wants to merge 11 commits intoCGAL:mainfrom
soesau:IO-VG_Import_Export-GF
Open

IO: add support for VG (VertexGroup) format#9436
soesau wants to merge 11 commits intoCGAL:mainfrom
soesau:IO-VG_Import_Export-GF

Conversation

@soesau
Copy link
Copy Markdown
Member

@soesau soesau commented Apr 17, 2026

Summary of Changes

Adding support for the vg format which can store segmented point clouds. The format stores a point cloud with optional normal and color components and a groups. Each group consists of a set of indices into the point cloud and type, color, label, parameters and child groups.

The format of child groups is unclear and will not be supported.

Official documentation:
https://abspy.readthedocs.io/en/latest/vertexgroup.html

Release Management

  • Affected package(s): Stream Support
  • License and copyright ownership: GeometryFactory

@soesau soesau added this to the 6.3-beta milestone Apr 17, 2026
@soesau soesau self-assigned this Apr 17, 2026
*/

template <typename PointRange, typename RegionRange, typename NamedParameters>
bool write_VG(std::ostream& os,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should that not be in a file named CGAL/IO/VG.h

@afabri
Copy link
Copy Markdown
Member

afabri commented Apr 20, 2026

Do we really need so big data in the VG format in Data/data/points? @sloriot

@afabri afabri requested a review from sloriot April 20, 2026 14:17
@soesau
Copy link
Copy Markdown
Member Author

soesau commented Apr 20, 2026

Do we really need so big data in the VG format in Data/data/points? @sloriot

Ah no. I can either make smaller ones, or I can just use Shape detection to generate vg data on the fly.

@afabri
Copy link
Copy Markdown
Member

afabri commented Apr 27, 2026

Do we really need so big data in the VG format in Data/data/points? @sloriot

Ah no. I can either make smaller ones, or I can just use Shape detection to generate vg data on the fly.

I guess it then also needs rewrite of history to not waste space @sloriot

Comment thread Stream_support/include/CGAL/IO/write_vg.h Outdated
Comment thread Stream_support/include/CGAL/IO/read_vg.h Outdated
@soesau soesau force-pushed the IO-VG_Import_Export-GF branch from adb2bdf to e0efe4b Compare April 29, 2026 14:09
@MaelRL MaelRL added CGAL I/O Not yet approved The feature or pull-request has not yet been approved. Small feature CHANGES.md not updated labels Apr 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants