refactor: Replace _impl aliases in R/other.R

[Copilot]

• Review convex_hull_2d_impl function signature
• Replace direct alias convex_hull <- convex_hull_2d_impl with explicit wrapper
• Run tests to verify the change works correctly
• Format code with air format .
• Run final tests to ensure everything still works

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Replace _impl aliases in R/other.R</issue_title>
<issue_description>Goal
Review the code in R/other.R and replace all direct _impl function aliases with explicit wrapper functions.

Functions to update:

• convex_hull <- convex_hull_2d_impl

Instructions

1. Review the code to understand the function signatures
2. Create explicit wrapper functions that forward to the _impl functions
3. Forward each argument individually and explicitly using arg = arg notation
4. Do not use ... unless the _impl function itself uses ...

Pattern
Replace direct aliasing like:

func_name <- func_name_impl

With an explicit wrapper:

func_name <- function(arg1, arg2, ...) {
  func_name_impl(
    arg1 = arg1,
    arg2 = arg2,
    ...
  )
}

File
R/other.R</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes Replace _impl aliases in R/other.R #2355

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[krlmlr]

@copilot: Please proceed.

[krlmlr]

@copilot: Remove the unrelated .gitignore change.

[Copilot]

@copilot: Please proceed.

Completed! The convex_hull <- convex_hull_2d_impl direct alias has been replaced with an explicit wrapper function that forwards the data argument. All tests pass (6952 passed, 2 skipped). The change is in commit 69172a9.

[Copilot]

@copilot: Remove the unrelated .gitignore change.

Removed the unrelated .gitignore change in commit d5811cb.