Skip to content

Add freehand polygon-drawing feature#25

Merged
mfl28 merged 6 commits intomasterfrom
feature/add-freehand-polygon-drawing
Aug 26, 2022
Merged

Add freehand polygon-drawing feature#25
mfl28 merged 6 commits intomasterfrom
feature/add-freehand-polygon-drawing

Conversation

@mfl28
Copy link
Owner

@mfl28 mfl28 commented Aug 21, 2022

  • Adds freehand polygon-drawing feature including (optional) topology-preserving simplification.
  • Adds EditorSettingsPane in settings to set polygon simplification algorithm settings.
  • Adds tests.

Closes #24 .

@codecov
Copy link

codecov bot commented Aug 21, 2022

Codecov Report

Merging #25 (9d71b64) into master (01e553d) will increase coverage by 0.08%.
The diff coverage is 86.66%.

@@             Coverage Diff              @@
##             master      #25      +/-   ##
============================================
+ Coverage     84.04%   84.13%   +0.08%     
- Complexity     1213     1281      +68     
============================================
  Files            86       89       +3     
  Lines          5309     5577     +268     
  Branches        477      505      +28     
============================================
+ Hits           4462     4692     +230     
- Misses          612      638      +26     
- Partials        235      247      +12     
Impacted Files Coverage Δ
...mfl28/boundingboxeditor/controller/Controller.java 72.15% <57.89%> (-0.05%) ⬇️
...om/github/mfl28/boundingboxeditor/ui/MainView.java 77.44% <70.37%> (-0.84%) ⬇️
...oundingboxeditor/ui/BoundingFreehandShapeView.java 74.66% <74.66%> (ø)
...8/boundingboxeditor/ui/WorkspaceSplitPaneView.java 82.07% <84.21%> (-0.08%) ⬇️
.../github/mfl28/boundingboxeditor/ui/EditorView.java 97.29% <88.88%> (-2.71%) ⬇️
...ndingboxeditor/ui/settings/EditorSettingsView.java 91.48% <91.48%> (ø)
...fl28/boundingboxeditor/ui/EditorImagePaneView.java 84.61% <92.75%> (+1.87%) ⬆️
...fl28/boundingboxeditor/ui/BoundingPolygonView.java 84.69% <94.20%> (+0.80%) ⬆️
.../boundingboxeditor/ui/BoundingPolygonTreeItem.java 62.50% <100.00%> (ø)
...l28/boundingboxeditor/ui/EditorSettingsConfig.java 100.00% <100.00%> (ø)
... and 9 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

81.1% 81.1% Coverage
2.9% 2.9% Duplication

@mfl28 mfl28 merged commit e73f4c8 into master Aug 26, 2022
@mfl28 mfl28 deleted the feature/add-freehand-polygon-drawing branch August 26, 2022 07:40
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.

Add polygon freehand-drawing

1 participant