-
Notifications
You must be signed in to change notification settings - Fork 244
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Core] Deprecate old projection methods #9024
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tteschemacher changes have been done accordingly. I also agree on adding the deprecation warning in the "standard" geometries only.
…d brep geometries
# Conflicts: # kratos/geometries/brep_curve.h # kratos/geometries/brep_curve_on_surface.h # kratos/geometries/brep_surface.h # kratos/geometries/nurbs_curve_geometry.h # kratos/geometries/nurbs_curve_on_surface_geometry.h # kratos/geometries/nurbs_surface_geometry.h
It seems that there is a |
@tteschemacher it seems that there are still some issues with the debug compilations. |
here we go. |
@rubenzorrilla @roigcarlo this is finished from my side and I think we should merge as soon as possible to avoid further conflicts. |
Completely agree, but I we need someone to review / approve it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Blame me if it breaks something 👍
Description
This PR deprecates the old
ProjectionPoint
method according to the discussion in #8491. So far only the "simple" geometries have been modified. @tteschemacher will proceed with the IGA-related geometries.ClosestPoint
method will be deprecated in an future PR.Changelog
Deprecation of the
ProjectPoint
method in favor of theProjectPointLocalToLocalSpace
andProjectPointGlobalToLocalSpace
.