Skip to content

Conversation

@Brookoko
Copy link
Owner

No description provided.

@codecov-io
Copy link

codecov-io commented Jun 11, 2019

Codecov Report

Merging #9 into dev will increase coverage by 4.68%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##                dev       #9      +/-   ##
============================================
+ Coverage     72.93%   77.62%   +4.68%     
- Complexity      176      223      +47     
============================================
  Files            31       34       +3     
  Lines           606      733     +127     
  Branches         44       62      +18     
============================================
+ Hits            442      569     +127     
  Misses          161      161              
  Partials          3        3
Impacted Files Coverage Δ Complexity Δ
...abs/introtoprogramming/lab5/scene/SceneObject.java 83.33% <100%> (+1.51%) 6 <1> (+1) ⬆️
...java/labs/introtoprogramming/lab5/tree/KDNode.java 100% <100%> (ø) 12 <12> (?)
...ntrotoprogramming/lab5/object/OptimizedObject.java 100% <100%> (ø) 3 <3> (?)
.../java/labs/introtoprogramming/lab5/object/Box.java 96.07% <100%> (+0.42%) 17 <2> (+2) ⬆️
...va/labs/introtoprogramming/lab5/object/Sphere.java 91.3% <100%> (+1.3%) 6 <1> (+1) ⬆️
...java/labs/introtoprogramming/lab5/tree/KDTree.java 100% <100%> (ø) 26 <26> (?)
.../labs/introtoprogramming/lab5/object/Triangle.java 100% <100%> (ø) 9 <1> (+2) ⬆️

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 066d43a...8d18ff1. Read the comment docs.

@Brookoko
Copy link
Owner Author

Tree

import java.util.List;

public class KDTree {
private static double MIN_NUMBER_OF_OBJECTS = 3;
Copy link
Collaborator

Choose a reason for hiding this comment

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

final

@Brookoko Brookoko merged commit d06a446 into dev Jun 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants