Skip to content

Conversation

gangatp
Copy link
Collaborator

@gangatp gangatp commented Sep 2, 2025

C# Binding Fix:

  • Add 'public' modifier to Wrapper class in buildbindingcsharp.go
  • Fixes accessibility issue where Wrapper class was internal instead of public
  • Aligns C# bindings with other languages that properly expose wrapper classes

Python Binding Enhancement:

  • Refactor out parameter handling in buildbindingpython.go
  • Separate 'out' and 'return' parameter processing logic
  • Add optional input parameters for out parameters with default None values
  • Improve backwards compatibility by maintaining out parameters in return values

@gangatp gangatp requested a review from Copilot September 2, 2025 07:00
@gangatp gangatp changed the base branch from master to develop September 2, 2025 07:00
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes binding issues for both C# and Python in the Automatic Component Toolkit (ACT). The changes improve C# wrapper class visibility and enhance Python out parameter handling for better backwards compatibility.

  • Adds 'public' modifier to C# Wrapper class to fix accessibility issues
  • Refactors Python out parameter handling to support optional input parameters with default None values
  • Maintains backward compatibility in Python bindings while improving parameter processing

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@spywo
Copy link

spywo commented Sep 5, 2025

LGTM

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