Skip to content

Try to robustify Straight Skeleton#2917

Closed
sloriot wants to merge 18 commits intoCGAL:masterfrom
sloriot:SS-robustify
Closed

Try to robustify Straight Skeleton#2917
sloriot wants to merge 18 commits intoCGAL:masterfrom
sloriot:SS-robustify

Conversation

@sloriot
Copy link
Copy Markdown
Member

@sloriot sloriot commented Mar 9, 2018

Some predicates used in the straight skeleton require several square roots. This robustifies one predicate and increases the precision of 2 constructions used in predicates.

@sloriot sloriot added this to the 4.12-beta2 milestone Mar 9, 2018
@sloriot sloriot requested a review from MaelRL March 9, 2018 13:52
@sloriot sloriot added the Not yet approved The feature or pull-request has not yet been approved. label Mar 9, 2018
#ifdef CGAL_USE_CORE

#ifdef CGAL_USE_GMPXX
inline CORE::BigFloat to_BigFloat( mpq_class const& n )
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.

Shouldn't we add stuff to CORE_coercion_traits.h instead?

@lrineau
Copy link
Copy Markdown
Member

lrineau commented Mar 12, 2018

@sloriot Do you still want that change in CGAL-4.12-beta2 or CGAL-4.12?

@sloriot sloriot modified the milestones: 4.12-beta2, 4.13-beta Mar 13, 2018
@lrineau
Copy link
Copy Markdown
Member

lrineau commented Jun 7, 2018

@sloriot Could you check the status of this PR?

@sloriot sloriot added the Stalled label Jun 7, 2018
@sloriot
Copy link
Copy Markdown
Member Author

sloriot commented Jun 7, 2018

Stalled

@lrineau lrineau removed this from the 4.13-beta milestone Jun 7, 2018
@lrineau lrineau added this to the Trash / Attic milestone Jan 7, 2019
@MaelRL MaelRL added Replaced and removed Not yet approved The feature or pull-request has not yet been approved. Work in progress labels Sep 22, 2020
@MaelRL
Copy link
Copy Markdown
Member

MaelRL commented Sep 22, 2020

Replaced by #4905.

@MaelRL MaelRL closed this Sep 22, 2020
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.

4 participants