Skip to content

[fitting] Add varifolds fitting#300

Open
Me-k-01 wants to merge 4 commits intoponcateam:masterfrom
Me-k-01:varifold
Open

[fitting] Add varifolds fitting#300
Me-k-01 wants to merge 4 commits intoponcateam:masterfrom
Me-k-01:varifold

Conversation

@Me-k-01
Copy link
Copy Markdown
Collaborator

@Me-k-01 Me-k-01 commented Mar 30, 2026

Readapted from pcloud-differential-estimation-benchmark

Changes made by this PR

  • Add varifolds fitting
  • Test varifolds

@Me-k-01 Me-k-01 self-assigned this Mar 30, 2026
@Me-k-01 Me-k-01 changed the title [Fitting] Add varifolds fitting [fitting] Add varifolds fitting Mar 30, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 30, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.09%. Comparing base (d74bced) to head (b1c2526).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #300      +/-   ##
==========================================
+ Coverage   94.07%   94.09%   +0.02%     
==========================================
  Files          83       83              
  Lines        3323     3322       -1     
  Branches      267      267              
==========================================
  Hits         3126     3126              
+ Misses        196      195       -1     
  Partials        1        1              
Files with missing lines Coverage Δ
Ponca/src/Fitting/weightKernel.h 100.00% <ø> (ø)

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread Ponca/src/Fitting/varifolds.h Outdated
Comment thread Ponca/src/Fitting/varifolds.h Outdated
Comment thread Ponca/src/Fitting/varifolds.h Outdated
Comment on lines +21 to +23
// rho prime is negative but the basket ignores negative weights
// (see Basket::addNeighbor()), so the opposite is returned here
// and Varifold::addLocalNeighbor() takes the opposite again
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Need to discuss this with @jmaleo

@Me-k-01 Me-k-01 force-pushed the varifold branch 10 times, most recently from c98faf4 to 456df0a Compare March 31, 2026 11:19
fauberva added 4 commits April 13, 2026 11:41
Signed-off-by: fauberva <florian.auberval@irit.fr>
- Move `VarifoldWeightKernel` to the Weight Kernel file
- Fix doc
- Add Varifolds paper to bibitex
- Fix PROVIDES

Signed-off-by: fauberva <florian.auberval@irit.fr>
- Add missing end of line in bibitex

Signed-off-by: fauberva <florian.auberval@irit.fr>
Signed-off-by: fauberva <florian.auberval@irit.fr>
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.

2 participants