Skip to content

Conversation

@lguerard
Copy link
Contributor

Fix #127 and add missing settings

* Fixed typo in 'lambda' parameter in interest_points_registration function.
* Removed unnecessary blank line for cleaner code.
* Added 'search_radius' and 'inlier_factor' parameters for better configuration.
@codecov
Copy link

codecov bot commented Jan 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 24%. Comparing base (ce66eef) to head (be55261).
⚠️ Report is 52 commits behind head on devel.

Additional details and impacted files
@@         Coverage Diff          @@
##           devel   #130   +/-   ##
====================================
- Coverage     25%    24%   -0%     
====================================
  Files         25     25           
  Lines       1711   1742   +31     
====================================
+ Hits         421    425    +4     
- Misses      1290   1317   +27     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

@ehrenfeu ehrenfeu added bug Something isn't working conventions Issue with formatting / naming / ... new-minor Change introduced in PR will require a new minor release. labels Jan 19, 2026
@ehrenfeu ehrenfeu linked an issue Jan 19, 2026 that may be closed by this pull request
@ehrenfeu ehrenfeu moved this to In progress in imcflibs Jan 19, 2026
@ehrenfeu ehrenfeu moved this from In progress to In review in imcflibs Jan 19, 2026
@ehrenfeu ehrenfeu added this to the 2.0.0 milestone Jan 19, 2026
@ehrenfeu ehrenfeu merged commit 842bd1f into imcf:devel Jan 19, 2026
4 of 5 checks passed
@github-project-automation github-project-automation bot moved this from In review to Done in imcflibs Jan 19, 2026
@lguerard lguerard deleted the fix/127-bdv_typo branch January 22, 2026 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working conventions Issue with formatting / naming / ... new-minor Change introduced in PR will require a new minor release.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Spelling mistake in bdv.interest_points_registration ?

2 participants