Skip to content

Conversation

DumDereDum
Copy link
Member

@DumDereDum DumDereDum commented Aug 12, 2020

This PR changes

/----[ WI ]----/

  • Reducing memory consumption:
    • Create new type (half)
    • Replace float with new type - TsdfType(int8_t):
      • CPU
      • GPU
      • INTRINSICS
      • HashTSDF
    • Replace int with uchar:
      • CPU
      • GPU
      • INTRINSICS
      • HashTSDF
  • TSDF optimization:
    • Speeded up the calculation of the pixNorm variable by incrementing this variable on the fixed number
  • HashTSDF optimization:
    • Vectorized interpolate function
    • Reduced the call to the at() function, acceleration of the getNormalVoxel() function by 33%, and the algorithm by 10%
  • Tests:
    • Create new struct "Settings" for performance test (perf_rgbd)
  • Other minor fixes

Pull Request Readiness Checklist

See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request

  • I agree to contribute to the project under OpenCV (BSD) License.
  • To the best of my knowledge, the proposed patch is not based on a code under GPL or other license that is incompatible with OpenCV
  • The PR is proposed to proper branch
  • There is reference to original bug report and related work
  • There is accuracy test, performance test and test data in opencv_extra repository, if applicable
    Patch to opencv_extra has the same branch name.
  • The feature is well documented and sample code can be built with the project CMake

@savuor savuor self-assigned this Sep 14, 2020
@savuor savuor self-requested a review September 14, 2020 10:55
@savuor
Copy link
Contributor

savuor commented Sep 14, 2020

👍

@alalek alalek merged commit b267fc3 into opencv:master Sep 17, 2020
@alalek
Copy link
Member

alalek commented Sep 20, 2020

Looks like there are failed tests in Debug build: #2675

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