Updated SolvePnPMethod enum.

[kasper-vv]

The SolvePnPMethod enum was missing the last 4 values. I have added these values and updated the descriptions/summaries.
SolvePnp with SolvePnPMethod.SQPNP was successfully tested on a real world scenario.

Summary by CodeRabbit

• Documentation
  • Improved XML documentation for the SolvePnPMethod solver method flags, including clearer descriptions for iterative (Levenberg–Marquardt), notes on DLS/UPNP limitations, and updated guidance on point ordering for IPPE_SQUARE.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 789c7aed-fe00-4d0f-b040-6cef34fd4c3a

📥 Commits

Reviewing files that changed from the base of the PR and between 57fe0e2 and 6c32c1a.

📒 Files selected for processing (1)

• src/OpenCvSharp/Modules/calib3d/Enum/SolvePnPFlags.cs

✅ Files skipped from review due to trivial changes (1)

• src/OpenCvSharp/Modules/calib3d/Enum/SolvePnPFlags.cs

---

📝 Walkthrough

Walkthrough

XML documentation comments in the SolvePnPMethod enum (SolvePnPFlags.cs) are rewritten. The Iterative entry's description is updated, and the comments for P3P, DLS, UPNP, AP3P, IPPE, IPPE_SQUARE, and SQPNP are replaced or expanded with revised bibliographic references, "broken implementation" warnings for DLS/UPNP, and detailed point-order requirements for IPPE_SQUARE. No enum values or runtime behavior are modified.

Changes

SolvePnPMethod XML Documentation

Layer / File(s)	Summary
SolvePnPMethod enum XML documentation src/OpenCvSharp/Modules/calib3d/Enum/SolvePnPFlags.cs	Rewrites the Iterative doc comment and replaces/expands documentation for all remaining enum members (P3P through SQPNP), including broken-implementation notes for DLS and UPNP and point-order requirements for IPPE_SQUARE.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• shimat/opencvsharp#1898: Both PRs modify SolvePnPMethod documentation in the same file, with #1898 adding the enum values and this PR updating their XML doc comments.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: updating documentation for the SolvePnPMethod enum with refined descriptions and new docstrings.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[kasper-vv]

Sorry, I didn't see that you had also made the changes yourself in #1898

[shimat]

Thank you!