Skip to content

Stream_support: Add write_MEDIT() for a tetrahedron soup - #9616

Open
afabri wants to merge 5 commits into
CGAL:mainfrom
afabri:Stream_support-write_medit-GF
Open

Stream_support: Add write_MEDIT() for a tetrahedron soup#9616
afabri wants to merge 5 commits into
CGAL:mainfrom
afabri:Stream_support-write_medit-GF

Conversation

@afabri

@afabri afabri commented Aug 27, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Summary of Changes

Add a function to write points and an indexed cell soup to a stream in the MEDIT format

Release Management

  • Affected package(s): Stream_support
  • Link to compiled documentation: link
  • License and copyright ownership: unchanged

@afabri
afabri requested a review from sloriot August 27, 2026 11:49
@afabri afabri self-assigned this Aug 27, 2026
@afabri afabri added this to the 6.3-beta milestone Aug 27, 2026
@afabri

This comment was marked as outdated.

Sign in to view
@github-actions

This comment was marked as outdated.

Sign in to view
@afabri

This comment was marked as outdated.

Sign in to view
@github-actions

This comment was marked as outdated.

Sign in to view
@afabri

This comment was marked as outdated.

Sign in to view
@github-actions

github-actions Bot commented Aug 27, 2026

Copy link
Copy Markdown

The documentation is built. It will be available, after a few minutes, here: https://cgal.github.io/9616/v0/Manual/index.html

sloriot
sloriot reviewed Aug 27, 2026
View reviewed changes
Comment thread Stream_support/include/CGAL/IO/MEDIT.h
bool write_MEDIT(std::ostream& os,
const PointRange& points,
const CellRange& finite_cells,
const std::vector<int>& subdomains)

@sloriot sloriot Aug 27, 2026 •
edited
Loading

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.

could be a named parameter with a property map returning 1 for all being the default

sloriot
sloriot reviewed Aug 27, 2026
View reviewed changes
Comment thread Stream_support/include/CGAL/IO/MEDIT.h Outdated
sloriot
sloriot reviewed Aug 27, 2026
View reviewed changes
Comment thread Stream_support/include/CGAL/IO/MEDIT.h Outdated
@afabri

afabri commented Aug 28, 2026

Copy link
Copy Markdown
Member Author

/force-build:v0

@github-actions

github-actions Bot commented Aug 28, 2026

Copy link
Copy Markdown

There was an error while building the doc:

/home/runner/work/cgal/cgal/Stream_support/doc/Stream_support/File_formats/Supported_file_formats.txt:561: warning: unable to resolve link to 'CGAL::IO::read_MEDIT(std::istream&, PointRange&, CellRange&, std::vector<int>&,bool)' for \link command
/home/runner/work/cgal/cgal/Stream_support/doc/Stream_support/File_formats/Supported_file_formats.txt:562: warning: unable to resolve link to 'CGAL::IO::write_MEDIT(std::ostream&, const PointRange&, const CellRange&, const std::vector<int>&)' for \link command
/home/runner/work/cgal/cgal/Stream_support/doc/Stream_support/IOstream.txt:508: warning: unable to resolve link to 'CGAL::IO::read_MEDIT(std::istream&, PointRange&, CellRange&, std::vector<int>&, bool)' for \link command
/home/runner/work/cgal/cgal/Stream_support/doc/Stream_support/IOstream.txt:509: warning: unable to resolve link to 'CGAL::IO::write_MEDIT(std::ostream&, const PointRange&, const CellRange&, const std::vector<int>&)' for \link command

https://github.com/CGAL/cgal/actions/runs/33160373477

@afabri

afabri commented Aug 28, 2026

Copy link
Copy Markdown
Member Author

/force-build:v0

@github-actions

github-actions Bot commented Aug 28, 2026

Copy link
Copy Markdown

The documentation is built. It will be available, after a few minutes, here: https://cgal.github.io/9616/v0/Manual/index.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sloriot sloriot sloriot left review comments

Assignees

@afabri afabri

Labels

Pkg::Stream_support Ready to be tested Under Testing

Projects

None yet

Milestone

6.3-beta

Development

Successfully merging this pull request may close these issues.

2 participants

@afabri @sloriot