Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Enable tests_2d and tests_io in AppVeyor. #2505

Merged
merged 4 commits into from
Oct 7, 2018
Merged

Enable tests_2d and tests_io in AppVeyor. #2505

merged 4 commits into from
Oct 7, 2018

Conversation

SergioRAgostinho
Copy link
Member

After the latest pushes, these should be working now.

@SergioRAgostinho SergioRAgostinho added this to the pcl-1.9.0 milestone Oct 1, 2018
@taketwo
Copy link
Member

taketwo commented Oct 2, 2018 

1: [ RUN      ] Edge.sobel
1: unknown file: error: SEH exception with code 0xc0000005 thrown in the test body.
1: [  FAILED  ] Edge.sobel (0 ms)

A single test failed on x86, but on x64 it succeeded.

@SergioRAgostinho
Copy link
Member Author

I was digging deep trying to replicate the problem on the VM and failing miserably. Failed to notice the x86/x64 detail. 👍

I fixed the other error regarding io_grabbers. It's also present in nix if you build PCL without VTK. I'll push both fixes at the same time though.

@taketwo taketwo merged commit 60151dc into PointCloudLibrary:master Oct 7, 2018
@SergioRAgostinho SergioRAgostinho deleted the enable-tests-appveyor branch October 9, 2018 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
module: ci module: test
Projects
None yet
Milestone
pcl-1.9.0
Development

Successfully merging this pull request may close these issues.
2 participants
@SergioRAgostinho @taketwo