Fix BridgeApi package imports and wrapper smoke tests#1
Open
alxdncn wants to merge 1 commit into
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Source / Evidence
Customer reported that
BridgeApi.pyraisesModuleNotFoundErrorforBridgeDataTypesunless users manually addbridge_python_sdktosys.path.Internal testing confirmed the
BridgeDataTypesimport issue is separate from the native Bridge crash and should be fixed in the Python SDK package import behavior.Summary
BridgeApi.pyto use package-relativeBridgeDataTypesimports when loaded asbridge_python_sdk.BridgeApi.src/bridge_python_sdkonsys.path.get_display_aspect()now calls the correctly bound native wrapper name.*_for_displayscalar wrappers now pass the debug flag expected by_scalar_call.Tests Run
python -m unittest tests.test_importspython -m compileall -q src\bridge_python_sdkManual Testing Notes
I did not run a full device-backed Bridge initialization test. The change was verified without instantiating
BridgeAPIagainst native Bridge runtime/device behavior.Docs Impact
None expected. This fixes SDK import/package behavior and related wrappers without changing documented usage.
Risk
Low. Changes are scoped to Python import/package behavior and lightweight wrapper call fixes.
Residual Risk
This does not address the separate native
bridge_inproc.dllload/init crash from the same customer thread.