Copy link Copy Markdown

Contributor

sarlinpe linked an issue

that may be closed by this pull request

Closed

Copy link Copy Markdown

Contributor

Is it possible to construct a test case that covers the issue? Thanks for the fix.

Copy link Copy Markdown

Contributor Author

Is it possible to construct a test case that covers the issue? Thanks for the fix.

Added some tests in the reconstruction class (we need to add some pipeline tests in the future). PTAL when time permits. The point_triangulator breaks in the current head so we should merge this PR ASAP : )

Copy link Copy Markdown

Contributor

Choose a reason for hiding this comment

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

Thanks, small suggestion. Otherwise, LGTM.

Merged