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.

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

Make testVoxelComplex faster #1451

Merged
merged 1 commit into from
Jan 19, 2020

Conversation

phcerdan
Copy link
Member

@phcerdan phcerdan commented Jan 9, 2020

Make testVoxelComplex faster

Reducing the amount of spels of Fixture_X by more than half.

Checklist

  • Unit-test of your feature with Catch.
  • [NA] Doxygen documentation of the code completed (classes, methods, types, members...)
  • [NA] Documentation module page added or updated.
  • New entry in the ChangeLog.md added.
  • No warning raised in Debug cmake mode (otherwise, Travis C.I. will fail).
  • All continuous integration tests pass (Travis & appveyor)

@phcerdan phcerdan changed the title # PR Description Make testVoxelComplex faster Jan 9, 2020
@dcoeurjo dcoeurjo self-assigned this Jan 9, 2020
@dcoeurjo
Copy link
Member

👌 thx
Could you please add a changelog entry for this one as well ?

Reducing the amount of spels of Fixture_X by more than half.
@phcerdan
Copy link
Member Author

Could you please add a changelog entry for this one as well ?

Done

@dcoeurjo
Copy link
Member

Thanks, merging

@kerautret there is a deadlink in the appveyor script.

@dcoeurjo dcoeurjo merged commit a47e72e into DGtal-team:master Jan 19, 2020
@phcerdan phcerdan deleted the faster_testVoxelComplex2 branch January 19, 2020 19:58
@kerautret
Copy link
Member

@dcoeurjo yes sorry the link is on a server that down for this weekend (it is my fault) it will be again live Monday 🤞🏻 so nothing to do ;)

@dcoeurjo
Copy link
Member

👌 thanks

@kerautret
Copy link
Member

@dcoeurjo it is working again ;)

dcoeurjo added a commit that referenced this pull request Oct 2, 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.

3 participants