Skip to content
This repository was archived by the owner on Jul 26, 2025. It is now read-only.

Conversation

EscapedGibbon
Copy link
Collaborator

No description provided.

@EscapedGibbon EscapedGibbon linked an issue Apr 19, 2023 that may be closed by this pull request
@codecov
Copy link

codecov bot commented Apr 19, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.01 🎉

Comparison is base (fc6c3d5) 98.47% compared to head (3fdca16) 98.49%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #309      +/-   ##
==========================================
+ Coverage   98.47%   98.49%   +0.01%     
==========================================
  Files         196      197       +1     
  Lines        5127     5173      +46     
  Branches     1114     1115       +1     
==========================================
+ Hits         5049     5095      +46     
  Misses         76       76              
  Partials        2        2              
Impacted Files Coverage Δ
src/roi/Roi.ts 99.61% <100.00%> (+<0.01%) ⬆️
src/roi/properties/getEllipse.ts 100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@lpatiny
Copy link
Member

lpatiny commented Apr 25, 2023

@EscapedGibbon Please add the following test case as well

0 1 0
0 1 0
0 1 0

Currently it yields to NaN. Could be improved in the future.

@EscapedGibbon EscapedGibbon requested a review from targos May 5, 2023 08:55
@targos
Copy link
Member

targos commented May 5, 2023

I can have a look, but in the end it's @lpatiny who knows best about this feature

@lpatiny lpatiny merged commit e332345 into main May 5, 2023
@lpatiny lpatiny deleted the 246-new-roi-property-ellipse branch May 5, 2023 12:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New ROI property: ellipse
3 participants