Skip to content

Moved libcosimc loading to _internal.py #16

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

davidhjp01
Copy link
Collaborator

@davidhjp01 davidhjp01 commented May 21, 2025

This pull request focuses on refactoring the codebase to replace the use of CosimLibrary.lib with the libcosimc() function across multiple files in the libcosimpy package. Additionally, it includes improvements to error handling and minor adjustments to the Conan build process.

Updated the loading process of libcosimc to avoid exception observed during a startup process.

Refactoring to use libcosimc():

  • Replaced CosimLibrary.lib with libcosimc() in all wrap_function calls across the following files:

  • Removed the CosimLibrary.py file, which previously handled the loading of the cosimc library, as it is no longer needed.

Improved error handling:

  • Enhanced the error message in CosimExecution.py when creating an execution from an OSP configuration file. The error message now includes details from get_last_error_message() for better debugging.

Conan build process adjustments:

  • Modified the hatch_build.py script to ensure config_settings is initialized as an empty dictionary if it is None.
  • Updated the Conan install command to include a specific build pattern (-b b2/*) in addition to the existing options.

@davidhjp01 davidhjp01 requested a review from restenb July 7, 2025 06:34
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.

1 participant