search

ORNL-CEES / DataTransferKit

A library for multiphysics solution transfer. ARCHIVED
https://datatransferkit.readthedocs.io/en/dtk-3.0/
BSD 3-Clause "New" or "Revised" License
47 stars 26 forks source link

Point search clean up #560

Closed Rombur closed 5 years ago

Rombur commented 5 years ago

Simplify code in PointSearch. Mainly move code around and return the output of functions instead of passing the output by reference. We might move the function sendDataAcrossNetwork to ArborX. There are not that many changes but because it looks by because of the indentations. You should hide the whitespace changes when reviewing this PR.

Rombur commented 5 years ago

ping

Note that jenkins is failing because it couldn't reach codecov.org

codecov-io commented 5 years ago

Codecov Report

Merging #560 into master will decrease coverage by <.01%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #560      +/-   ##
==========================================
- Coverage    95.9%   95.89%   -0.01%     
==========================================
  Files          86       86              
  Lines        6027     6017      -10     
==========================================
- Hits         5780     5770      -10     
  Misses        247      247
Impacted Files Coverage Δ
...ckages/Discretization/src/DTK_PointSearch_decl.hpp 100% <ø> (ø) :arrow_up:
...ackages/Discretization/src/DTK_PointSearch_def.hpp 99.23% <100%> (-0.03%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update d0a9348...c1a8c6e. Read the comment docs.